[ https://issues.apache.org/jira/browse/BOOKKEEPER-1045?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996341#comment-15996341 ]
ASF GitHub Bot commented on BOOKKEEPER-1045: -------------------------------------------- Github user asfgit closed the pull request at: https://github.com/apache/bookkeeper/pull/135 > Execute tests in different JVM processes > ---------------------------------------- > > Key: BOOKKEEPER-1045 > URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1045 > Project: Bookkeeper > Issue Type: Test > Reporter: Matteo Merli > Assignee: Matteo Merli > Priority: Minor > Fix For: 4.5.0 > > > The current Maven Surefire configuration is using: > {code:xml} > <forkMode>always</forkMode> > {code} > This is a deprecated config and apparently it's not creating new processes > for each test as intended. > Currently the tests are leaking a big number of files and threads due to > several reasons: > * Tests that instantiate bookies and call shutdown() without calling start() > before are creating and initializing the ledger storage but not closing it, > leaking threads and several fds > * ZooKeeperClient sometimes doesn't shutdown the zk handle if the test > completes too quickly, leaking sockets. > * Several tests are passing bad config, so the bookie/client start gets > exception (on purpose) and then doesn't clean up some partial objects. > * ... > That make running the test suite to be dependent on ulimit of the machine. > Until we can fix (almost) all the test to do proper cleanup, we should make > maven to run tests in separated processes. -- This message was sent by Atlassian JIRA (v6.3.15#6346)