I just discovered a test case that breaks with the new prepared statement caching support.  I haven't taken the time to chase it down yet.
 
But it got me wondering if this is something we should turn off for 2.2.1.  I turned it off in my test environment by changing the method "SessionScope.hasPreparedStatementFor(String sql)" so that it always return false.  Maybe there's a better way to disable it?
 
IMO this seems like a risky thing to add in a maintenance release.  It also seems unneccessary - as most decent JDBC drivers already support caching prepared statements automatically.
 
Thoughts???
Jeff Butler
 

Reply via email to