Did this make it into the 1.2.1 release? Or since it was committed to the trunk will it have to wait for the 1.3 version?
-Dave On 1/18/07, Stefan Guggisberg <[EMAIL PROTECTED]> wrote:
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 >