On Thu, 29 Jan 2026 14:32:59 GMT, Roger Riggs <[email protected]> wrote:
>> test/jdk/java/lang/ProcessBuilder/PipelineLeaksFD.java line 246:
>>
>>> 244: pids.forEach(pid -> command.append(",").append(pid));
>>> 245: System.out.println("Running lsof command: " + command);
>>> 246: try (Process p = new ProcessBuilder(command.toString().split("
>>> "))
>>
>> The purpose of NOT using -p was to be able to identify the other processes
>> that may be linked by their file descriptors/handles if the case of a
>> failure.
>> Without a more complete list of file descriptors, the diagnostic information
>> is not available to track down the other process.
>> I'd prefer that the timeout be increased.
>
> Having just updated this test, I would have been interested in fixing any
> follow on problems myself.
Do you mean I should close this PR, and you will create a new one.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29478#discussion_r2744210430
