Does anyone have time to review PR #3123 for me?

GEODE-6301: Use ThreadInfo.toString in ExecutorServiceRule.dumpThreads
https://github.com/apache/geode/pull/3123

This PR changes ExecutorServiceRule.dumpThreads to use
java.lang.management.ThreadInfo.toString() which includes lock monitors and
synchronizers.

The only usage of this so far is the regression test that I attached
to GEODE-6255. When the test reproduces the hang, it uses
ExecutorServiceRule.dumpThreads() in the assertion failure output to show
the stacks of the two threads that are deadlocked.

Thanks,
Kirk

Reply via email to