Subject | Re: [Firebird-Java] Re: Memory Leak in client-java - FIXED! |
---|---|
Author | Lars George |
Post date | 2002-10-24T21:11:26Z |
Roman,
Just to confirm...
method with (!) the fix in it, i.e. the extra line (line no 692):
TransactionStatementMap.remove(tr);
fixes the leak we found. It sounded a bit as if you want to search for that
leak yourself. Sorry for being picky, I just wanted to clarify that so you
do not have to spent you precious time on the search again.
Thanks,
Lars
Just to confirm...
> > Since it is private and used only in two spots (commit andYou are aware that what I sent to the list is already a version of that
> > rollback) it seems the best fit to implement the fix. Could someone
> > do a code review on this and check the fix into CVS please?
>
> Yes, this is in my todo list before the RC2 is released.
method with (!) the fix in it, i.e. the extra line (line no 692):
TransactionStatementMap.remove(tr);
fixes the leak we found. It sounded a bit as if you want to search for that
leak yourself. Sorry for being picky, I just wanted to clarify that so you
do not have to spent you precious time on the search again.
Thanks,
Lars