You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/10/27 15:04:02 UTC

[GitHub] [activemq-artemis] gemmellr commented on a diff in pull request #4273: ARTEMIS-4076: add JDK 19 run to CI jobs

gemmellr commented on code in PR #4273:
URL: https://github.com/apache/activemq-artemis/pull/4273#discussion_r1007001674


##########
artemis-unit-test-support/src/main/java/org/apache/activemq/artemis/utils/ThreadLeakCheckRule.java:
##########
@@ -244,7 +244,7 @@ private boolean isExpectedThread(Thread thread) {
          return true;
       } else if (threadName.contains("Attach Listener")) {
          return true;
-      } else if ((javaVendor.contains("IBM") || isSystemThread) && threadName.equals("process reaper")) {
+      } else if ((javaVendor.contains("IBM") || isSystemThread) && (threadName.equals("process reaper") || threadName.startsWith("process reaper (pid"))) {

Review Comment:
   It isnt always in the name, it comes and goes.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org