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