incubator-jena-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andy Seaborne <>
Subject Re: svn commit: r1157790 - /incubator/jena/Experimental/TxTDB/trunk/src/main/java/com/hp/hpl/jena/tdb/transaction/
Date Mon, 15 Aug 2011 19:53:18 GMT

On 15/08/11 13:57, Paolo Castagna wrote:
> Hi Andy,
> NodeTableTrans has an empty implementation for the close() method.
> Shouldn't we call close on journal (i.e. journal.close()) and, possibly,
> on base (i.e. base.close())?
> I can easily run into troubles using TestTransSystem with an on-disk
> location instead of an in-memory one. After a while, you hit the
> problem: "Too many open files".
> Calling journal.close() seems to fix it.
> Paolo

Yes - it does seem to fix it although some tests are affected (the test 
look wrong).  The intent was (I think) to allow for reusable transaction 
objects to avoid the churn cost of java objects but it's a false economy.


View raw message