RFR 9: 8132394 : (process) ProcessBuilder support for a pipeline of processes

Paul Sandoz paul.sandoz at oracle.com
Mon Nov 9 12:18:58 UTC 2015


> On 9 Nov 2015, at 12:20, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> 
> 
> On 05/11/2015 21:56, Roger Riggs wrote:
>> Please review the new ProcessBuilder.startPipeline API, implementation, and tests.
>> 
>> :
>> 
>> javadoc of ProcessBuilder:  only startPipeline is new:
>>   http://cr.openjdk.java.net/~rriggs/pipedoc/
>> 
> I skimmed over the javadoc and it looks quite good.
> 
> The method name seems okay, I initially assumed it would be something like startProcesses but that doesn't convey how they are arranged.
> 
> Using varargs seem reasonable for the examples like you have in the javadoc but there may be other usages where this might be annoying.

Good point. In light of the imminent (getting Stuart stressed here :-) ) arrival of the convenience collection factories it may be better to accept a List<ProcesssBuilder>, especially so since this is only really useful for two or more process builders.

Paul.





More information about the core-libs-dev mailing list