Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Match RI StackTraceElement format #2970

Merged
merged 1 commit into from
Sep 20, 2018

Conversation

JasonFengJ9
Copy link
Member

Match RI StackTraceElement format

Moved module name to the beginning of output;
Removed module version info.

Manually verified that pConfig still compiles.
Now OpenJ9 has following StackTraceElement format:

java.base/java.security.AccessController.throwACE(AccessController.java:176)
java.base/java.security.AccessController.checkPermissionHelper(AccessController.java:237)
java.base/java.security.AccessController.checkPermission(AccessController.java:373)
java.base/java.lang.SecurityManager.checkPermission(SecurityManager.java:322)
java.base/java.lang.SecurityManager.checkPropertyAccess(SecurityManager.java:1066)
java.base/java.lang.System.getProperty(System.java:450)
java.base/java.lang.System.getProperty(System.java:419)
atestse.java_lang.TestSTE.testAll(TestSTE.java:11)
atestse.java_lang.TestSTE.main(TestSTE.java:6)

While RI JDK11 ea b27 has following:

java.base/java.security.AccessControlContext.checkPermission(AccessControlContext.java:472)
java.base/java.security.AccessController.checkPermission(AccessController.java:895)
java.base/java.lang.SecurityManager.checkPermission(SecurityManager.java:322)
java.base/java.lang.SecurityManager.checkPropertyAccess(SecurityManager.java:1066)
java.base/java.lang.System.getProperty(System.java:810)
atestse.java_lang.TestSTE.testAll(TestSTE.java:11)
atestse.java_lang.TestSTE.main(TestSTE.java:6)

close:#2922

Reviewer: @pshipton
FYI: @DanHeidinga

Signed-off-by: Jason Feng fengj@ca.ibm.com

Moved module name to the beginning of output;
Removed module version info.

Signed-off-by: Jason Feng <fengj@ca.ibm.com>
@pshipton
Copy link
Member

jenkins test sanity zlinux jdk11

@pshipton
Copy link
Member

jenkins test extended zlinux jdk11

@pshipton pshipton merged commit 1bbf4a1 into eclipse-openj9:master Sep 20, 2018
@JasonFengJ9 JasonFengJ9 deleted the matchriste branch February 19, 2019 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stack trace elements from JDK classes should include module name on JDK 9+
2 participants