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