RFR: 8153133: Thread.dumpStack() can use StackWalker [v3]

Jaikiran Pai jpai at openjdk.java.net
Tue Nov 9 01:57:43 UTC 2021


On Mon, 8 Nov 2021 19:49:06 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> I'm uncomfortable with this change, does the change have any benefit?

To me, the initial appeal of this change was to use a more natural API instead of creating an `Exception` instance and printing the stacktrace from it. Performance wise, I hadn't run any numbers. But having to fallback to using the Exception instance in this above case, that too by doing an almost hackish check, does indeed remove any kind of benefit to this change.

Like Mandy suggests I will go ahead and withdraw this PR and close the linked issue.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6292


More information about the core-libs-dev mailing list