On Fri, 10 Mar 2023 00:01:51 GMT, Roger Riggs <rri...@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:
> 
>   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

Reply via email to