RFR: 8301627: System.exit and Runtime.exit debug logging [v6]
Chris Hegarty
chegar at openjdk.org
Mon Feb 20 11:03:27 UTC 2023
On Fri, 17 Feb 2023 17:27:50 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> It can be difficult to find the cause of calls to `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java runtime exits.
>> The status value and stack trace are logged using the System Logger named `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`.
>
> Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
>
> - Move test to the java/lang/RuntimeTests directory.
> Added implNote to System.exit.
> A few javadoc updates for review comments.
> - Merge branch 'master' into 8301627-log-system-exit
> - Improve implNote for Runtime.exit() with review suggestions.
> - Correct System.getLogger link
> - Add an @implNote to Runtime.exit to describe the java.lang.Runtime logging.
> - Added try/catch around lookup of logger so exceptions do not prevent System.exit.
> Added test case with console logger (when java.util.logging) not present.
> Removed @implNote tag its not appropriate in implementation javadoc.
> Still looking into when and where the log configuration should be described.
> - Locate the System logger before taking the shutdown lock
> - Add logging of calls to Runtime.exit to the system logger "java.lang.Runtime".
I left one trivial comment relating to a typo, but otherwise LGTM.
src/java.base/share/classes/java/lang/Runtime.java line 160:
> 158: *
> 159: * @implNote
> 160: * If the {@linkplain System#getLogger(String) the system logger} for {@code java.lang.Runtime}
Trivially, remove the double "the the" here.
-------------
Marked as reviewed by chegar (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12517
More information about the core-libs-dev
mailing list