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

2022-07-21 Thread Lance Andersen
On Thu, 21 Jul 2022 16:01:01 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 [v8]

2022-07-21 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 by

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 by

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 sin

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

2022-07-12 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 by

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

2022-07-11 Thread Jaikiran Pai
On Mon, 11 Jul 2022 15:31:46 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 [v4]

2022-07-11 Thread Jaikiran Pai
On Mon, 11 Jul 2022 15:31:46 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 [v4]

2022-07-11 Thread Alan Bateman
On Mon, 11 Jul 2022 15:31:46 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 [v4]

2022-07-11 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 by

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

2022-07-11 Thread Roger Riggs
On Sat, 9 Jul 2022 02:43:55 GMT, David Schlosnagle wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Cleanup of PipelineLeaksFD test improving error messages and source >> cleanup. > > test/jdk/java/lang/ProcessBuild

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

2022-07-11 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 by

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

2022-07-08 Thread David Schlosnagle
On Fri, 8 Jul 2022 16:24: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 e

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

2022-07-08 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 by

Re: RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline

2022-07-08 Thread Jaikiran Pai
On Thu, 7 Jul 2022 19:08:35 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 each pr

Re: RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline

2022-07-08 Thread Roger Riggs
On Fri, 8 Jul 2022 13:27:42 GMT, Alan Bateman 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

2022-07-08 Thread Alan Bateman
On Thu, 7 Jul 2022 19:08:35 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 each pr

RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline

2022-07-08 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 by the child p