RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v6]
Alan Bateman
alanb at openjdk.org
Tue Mar 14 15:43:58 UTC 2023
On Fri, 10 Mar 2023 21:34:56 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Runtime.exec and ProcessBuilder.start methods create a new operating system process with the program and arguments. Many applications configure a logging subsystem to monitor application events. Logging a process start message with the program, arguments, and stack trace can identify the caller and purpose.
>> Logging the process start event is complementary to the process start event generated for JFR (Java Flight Recorder).
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>
> Reword @implNote to indicate logging can be enabled. (But is not by default)
Marked as reviewed by alanb (Reviewer).
src/java.base/share/classes/java/lang/ProcessBuilder.java line 194:
> 192: {
> 193: // Lazily and racy initialize when needed, racy is ok, any logger is ok
> 194: private static System.Logger LOGGER ;
Minor nit, there's a space before the semi-colon.
src/java.base/share/classes/java/lang/ProcessBuilder.java line 205:
> 203: System.Logger logger = System.getLogger("java.lang.ProcessBuilder");
> 204: return logger.isLoggable(System.Logger.Level.DEBUG) ? logger : null;
> 205: }
Is initLogger left over from a previous iteration?
-------------
PR: https://git.openjdk.org/jdk/pull/12862
More information about the core-libs-dev
mailing list