On 1/18/07, Olivier Dony <[EMAIL PROTECTED]> wrote:
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!
great, i committed the fix in trunk (r497392).
cheers
stefan
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
