On Fri, 10 Mar 2023 00:01:51 GMT, Roger Riggs <[email protected]> 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:
>
> Reorder pid, directory, args in logged message.
> Cleanup argument lists in tests.
src/java.base/share/classes/java/lang/Runtime.java line 353:
> 351: *
> 352: * @implNote
> 353: * The new process creation is logged by {@link
> ProcessBuilder#start()}.
I agree it is useful to have an implNote here but I don't think it words to
just say that it is logged by PB.start. Instead I think it should say that the
process created is logged, see PB.start for details.
-------------
PR: https://git.openjdk.org/jdk/pull/12862
