Both myself and another user found this change to make a significant performance difference (hundreds of times faster).

This should work.  I'd be interested in learning more about what it broke. 

Cheers,
Clinton

On 10/5/06, Jeff Butler <[EMAIL PROTECTED]> wrote:
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