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

Posted to dev@geode.apache.org by Kirk Lund <kl...@apache.org> on 2019/02/04 18:25:32 UTC

Review request for PR #3123

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