threez/activerecord-jdbc-adapter — comment on commit 7689abec0933a8f6c18f6f4d6cbb0270dabf00a4
File:  line 0
By gfmurphy:

*This change* will cause a failure in the arjdbc test suite (simple.rb) since it spec's the ActiveRecord::JDBCError should be raised in this case; however, I don't think this is correct as the active record test suite implies that raising ActiveRecord::StatementInvalid is the correct behavior.

I haven't deduced the historical reasons for raising the JDBCError instead of StatementInvalid. It's raised in the RubyJdbcConnection object so it affects more than the jdbcsqlite3 adapter.

Personally I think that this should be changed for all the adapters in the project and the test suite should be updated to test for the StatementInvalid error.