Re: RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline [v7]

2022-07-21 Thread Jaikiran Pai
On Mon, 18 Jul 2022 17:53:16 GMT, Roger Riggs wrote: >> The `ProcessBuilder.pipelineStart()` implementation does not close all of >> the file descriptors it uses to create the pipeline of processes. >> >> The process calling `pipelineStart()` is creating the pipes between the >> stages. >> As

Re: RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline [v7]

2022-07-18 Thread Roger Riggs
> The `ProcessBuilder.pipelineStart()` implementation does not close all of the > file descriptors it uses to create the pipeline of processes. > > The process calling `pipelineStart()` is creating the pipes between the > stages. > As each process is launched, the file descriptor is inherited

Re: RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline [v7]

2022-07-18 Thread Roger Riggs
On Fri, 8 Jul 2022 13:29:12 GMT, Jaikiran Pai wrote: >> Roger Riggs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 10 additional >> commits