On 1/19/07, David Johnson <[EMAIL PROTECTED]> wrote:
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?
no and no. as jukka said it'll be part of a next patch release which will follow the first (i.e. main) release 1.2.1 probably soon after. cheers stefan
-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 > > >
