Re: RFR 8129120 Terminal operation properties should not be back-propagated to upstream operations

2015-06-22 Thread Chris Hegarty
The source changes look good to me Paul. I only skimmed over the test changes, nothing popping out. -Chris. On 19 Jun 2015, at 17:07, Paul Sandoz wrote: > On Jun 19, 2015, at 2:04 PM, Paul Sandoz wrote: >> There are two main aspects to the webrev: >> >> 1) remove the back propagation logic,

Re: RFR 8129120 Terminal operation properties should not be back-propagated to upstream operations

2015-06-19 Thread Paul Sandoz
On Jun 19, 2015, at 2:04 PM, Paul Sandoz wrote: > There are two main aspects to the webrev: > > 1) remove the back propagation logic, which simplifies the preparation of the > pipeline for parallel execution and also cleans up a certain aspect of > toArray optimisation that i had been meaning g

RFR 8129120 Terminal operation properties should not be back-propagated to upstream operations

2015-06-19 Thread Paul Sandoz
Hi, http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8129120-flag-back-propagation/webrev/ I am a little red in the face (of the embarrassed kind) about this fix. We found an issue reported on stack overflow whereby a stream can report incorrect results. I may be wrong but this might be the first r