michaelandrepearce commented on a change in pull request #2517: [ARTEMIS-2171]: ThreadPoolExecutor leak under SM due to lack of privileged block. URL: https://github.com/apache/activemq-artemis/pull/2517#discussion_r250970503
########## File path: artemis-commons/src/main/java/org/apache/activemq/artemis/core/server/NetworkHealthCheck.java ########## @@ -157,7 +157,12 @@ public NetworkHealthCheck parseURIList(String addressList) { @Override protected ActiveMQThreadFactory getThreadFactory() { - return new ActiveMQThreadFactory("NetworkChecker", "Network-Checker-", false, getThisClassLoader()); Review comment: So we need this converted to a test, e.g. integration test case, with this PR which shows this, given a certain configuration and a way to automate the detection. There is an issue of ActiveMQThreadFactory that was introduced in 2.4.0, that meant threads would be created upto the max pool size, instead of re-using threads, just only recently fixed in master, so a test would be a good way of seeing if your issue is fixed by that or different, and also so any fix needed is working. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services