RFR: 8263729: [test] Extend time to wait before destroying child in ProcssBuilder Basic test [v3]

Ioi Lam iklam at openjdk.java.net
Fri Mar 19 18:13:40 UTC 2021


On Fri, 19 Mar 2021 17:17:02 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Intermittent failures on Windows in a test of destroying the child warrant extending the time the parent waits after starting the child before destroying the child.
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Correct redirect of stdout

LGTM. Just a small nit.

test/jdk/java/lang/ProcessBuilder/Basic.java line 2140:

> 2138:                 List<String> childArgs = new ArrayList<>(javaChildArgs);
> 2139:                 final ProcessBuilder pb = new ProcessBuilder(childArgs);
> 2140:                 {

I was a little confused until I looked up the JavaDoc of ProcessBuilder.

Maybe add a comment like "ProcessBuilder does NOT make a copy of childArgs so it's OK to modify it below"?

-------------

Marked as reviewed by iklam (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3049


More information about the core-libs-dev mailing list