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.
Andy
