On Jan 17, 2007, at 6:19 PM, Stefan Guggisberg wrote:
doh! probably the simplest fix would be to remove line 783 in
DatabasePersistenceManager.java and line 1010 in
DatabaseFileSystem.java,
i.e. the following stmt:
preparedStatements.clear();
if you have a chance to successfully test this i'll commit the
change asap.
Ah you're right, there's no real need to clear the map anyway, as the
statements will all get replaced in initPreparedStatements() after a
successful reconnection.
Alright, so we've freshly rebuilt tag 1.2-rc2 with the lines you
mention commented out.
After testing a little bit, this fix seems to work fine for us, and
it can now reconnect correctly even if the database server is down
for a while!
I say ship it! ;-)
Would it be possible to still include a fix for this in 1.2?
i guess jukka will have to decide this.
I do hope it gets included, as the change is really minor and makes
the solution a lot more robust.
(Apparently my boss would sleep better knowing we're using an
official jackrabbit release instead of a modified version... go
figure...)
Thank you!
Olivier
