Connection errors are opaque

chop-dbhi / dataexpress

[NOT MAINTAINED] DataExpress is a simple, Scala-based cross database ETL toolkit supporting Postgres, MySql, Oracle, SQLServer, and Sqlite

http://dataexpress.research.chop.edu

BSD 2-Clause "Simplified" License

72 stars 9 forks source link

Connection errors are opaque #35

Closed mitalia closed 11 years ago

mitalia commented 11 years ago

If a user supplies a malformed jdbc URI, the error thrown is opaque. There needs to be some better detection of error conditions. The following should be done:

[x] Wrap the connection setup in a try/catch to provide more verbose errors
[ ] This class should be more Scala-like and should have the connection be an Option type to force handling of nulls
[x] When logging is implemented, the setup and teardown of the connection should be logged

mitalia commented 11 years ago

Fixed the original intent of this bug. The Option type on the connection would be easier if it were done after the tests were refactored. Will open another ticket for that.