SqlBackend should use an Option type for connections

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

SqlBackend should use an Option type for connections #39

Open mitalia opened 11 years ago

mitalia commented 11 years ago

The SqlBackend is more java-like than Scala-like. It probably should use the Option type for connections instead of nulls. That being said, it may be necessary to hold off on this conversion until the unit tests are in better shape.