add unknown database support, or specificaly oracle support

[GoogleCodeExporter]

What steps will reproduce the problem?
try to migrate with unknown database (i.e. oracle) jdbc url (i.e.
jdbc:oracle:thin:scott/tiger@//myhost:1521/orcl), with specified driver
class -- i.e no need for driver class auto guessing; I still receive
NullPointerException

What is the expected output? What do you see instead?
successful migration, instead of NPE:
java.lang.NullPointerException
        at
com.carbonfive.db.jdbc.ScriptRunnerFactory.createScriptRunner(ScriptR
unnerFactory.java:9)
        at
com.carbonfive.db.migration.SQLScriptMigration.migrate(SQLScriptMigration.java:2
9)
        at
com.carbonfive.db.migration.DataSourceMigrationManager$5.doInConnection(DataSour
ceMigrationManager.java:147)
        at
org.springframework.jdbc.core.JdbcTemplate.execute(JdbcTemplate.java:342)

What version of the product are you using? On what operating system?
latest 0.9.3 snapshot, SUSE Linux

Please provide any additional information below.
I've attached a patch that adds support for oracle, and for unknown
database to eliminate the NPE

Original issue reported on code.google.com by zregv...@gmail.com on 21 Oct 2008 at 9:48

Attachments:

• patch.txt

[GoogleCodeExporter]

Thanks for supplying the patch!  I'll fold this in for 0.9.4.

Original comment by christia...@gmail.com on 21 Oct 2008 at 3:32

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by christia...@gmail.com on 21 Oct 2008 at 3:32

[GoogleCodeExporter]

Fixed in latest 0.9.4-SNAPSHOT.  Thanks!

Original comment by christia...@gmail.com on 25 Oct 2008 at 7:27

• Changed state: Fixed

[GoogleCodeExporter]

wohoa! that was fast, thanks!

Original comment by zregv...@gmail.com on 26 Oct 2008 at 8:09