Github user gaohoward commented on the issue:
https://github.com/apache/activemq-artemis/pull/1800
@clebertsuconic re: "test1 failed.. leaked.. than test2 failed because
there's a thread leak... if you don't report the leak from 1.. you won't know
that test2 could fail later.."
In that case test1 should be looked at first. after test1 is corrected the
thread check rule kicks in to make sure it doesn't leak, therefore won't
pollute test2.
The harder case is test1 passed..leaked.. then test2 failed becasue of the
leak. Even worse case is that test1 passed leaked and tests runs on until
test(n) failed due to the test1's leak. This PR doesn't allow that happen
because it checks every passing test for thread leaks.
---
