<p dir="ltr">Stateful programming has its issues but that ship has already sailed (in Java).</p>
<p dir="ltr">The programs where these new expressions will live are full of state...</p>
<p dir="ltr">With the introduction of streams, programmers and refactoring tools will be introducing the cool new expressions into existing code. (forEach is the groovy guy&#39;s for loop, right?)</p>
<p dir="ltr">I don&#39;t want to create danger zones in the code where these transformations are accidents waiting to happen. Also think of the code maintainers trying to determine, as they are enhancing and debugging the code, where they are allowed to add state.</p>

<p dir="ltr">Before, the existence of parallel() created a danger zone, but sequential() restored safety. That&#39;s an easy rule to understand.</p>
<p dir="ltr">BTW, what are the rules in Scala and groovy?</p>
<p dir="ltr">Joe</p>
<div class="gmail_quote">On Mar 22, 2013 9:33 AM, &quot;Brian Goetz&quot; &lt;<a href="mailto:brian.goetz@oracle.com">brian.goetz@oracle.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The problem with stateful lambdas is that, unless one block of code has control over the entire pipeline, it is an accident waiting to happen.<br>
<br>
Let&#39;s say you receive a stream as a parameter:<br>
<br>
  void foo(Stream s) { ... }<br>
<br>
and you want to do something that requires a stateful mapper:<br>
<br>
  void foo(Stream s) {<br>
      s.map(... stateful ...)...<br>
  }<br>
<br>
That&#39;s a bug already.  Because you don&#39;t know that the stream you were passed in is sequential.  But I doubt that people will remember, even most of the time, they need to do:<br>
<br>
      s.sequential().map(... stateful ...)...<br>
<br>
instead.  Won&#39;t happen.<br>
<br>
Stateful lambdas introduce the need for non-modular reasoning about stream pipelines (who created this? who will consume this? in what state was it created?).  And, it has all the same self-deception problems as thread-safety.  People convince themselves &quot;I don&#39;t need to think about synchronization because no one will ever use this object concurrently.&quot;<br>

<br>
So, while I sympathize with the desire to let people say &quot;I know that this entire stream pipeline has been carefully controlled such as to not have statefulness distort its results&quot;, I think in reality, this will quickly turn into &quot;statefulness is OK&quot; in most people&#39;s minds.  With the attended inevitable foot-shooting.<br>

<br>
On 3/21/2013 9:57 PM, Joe Bowbeer wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I&#39;m traveling now and won&#39;t be able to respond promptly but this topic<br>
has been raised a couple of times already. Feel free to copy and paste<br>
my response from previous discussions:)<br>
<br>
Rephrasing, I&#39;m OK with non-interference but I object to banning<br>
stateful in sequential ops.<br>
<br>
I think there should be a one-one correspondence between any for loop<br>
and a sequential forEach.<br>
<br>
Can you compare your restrictions with those in Scala and Groovy? Scala<br>
in particular, because it is more strictly defined, and I&#39;m pretty sure<br>
I&#39;ve combined stateful expressions with functional forms in Scala, to<br>
good effect. (One of the benefits of being multi-paradigmatic?)<br>
<br>
In addition, I&#39;m wary of the new form of forEach. If anything, I&#39;d like<br>
its name to be simpler, e.g., each, not longer.<br>
<br>
Joe<br>
<br>
On Mar 21, 2013 3:48 PM, &quot;Brian Goetz&quot; &lt;<a href="mailto:brian.goetz@oracle.com" target="_blank">brian.goetz@oracle.com</a><br>
&lt;mailto:<a href="mailto:brian.goetz@oracle.com" target="_blank">brian.goetz@oracle.com</a><u></u>&gt;&gt; wrote:<br>
<br>
    Doug and I have been revisiting sequential() and parallel().  I<br>
    think there&#39;s a nice simplification here.<br>
<br>
    The original motivation for sequential() was because we originally<br>
    had .into(collection), and many collections were not thread-safe so<br>
    we needed a means to bring the computation back to the current<br>
    thread.  The .sequential() method did that, but it also brought back<br>
    a constraint of encounter ordering with it, because if people did:<br>
<br>
        stuff.parallel().map(...).__<u></u>sequential().into(new ArrayList&lt;&gt;());<br>
<br>
    not respecting encounter order would violate the principle of least<br>
    astonishment.<br>
<br>
    So the original motivation for sequential() was &quot;bring the<br>
    computation back to the current thread, in order.&quot;  This was doable,<br>
    but has a high price -- a full barrier where we buffer the contents<br>
    of the entire stream before doing any forEach&#39;ing.<br>
<br>
    Most of the time, sequential() only appears right before the<br>
    terminal operation.  But, once implemented, there was no reason to<br>
    constrain this to appear right before the forEach/into, so we didn&#39;t.<br>
<br>
    Once we discovered a need for .parallel(), it made sense it be the<br>
    dual of .sequential() -- fully unconstrained.  And again, the<br>
    implementation wasn&#39;t *that* bad -- better than .sequential().  But<br>
    again, the most desirable position for .parallel() is right after<br>
    the source.<br>
<br>
    Then we killed into() and replaced it with reduction, which is a<br>
    much smarter way of managing ordering.  Eliminating half the<br>
    justification for .sequential().<br>
<br>
    As far as I can tell, the remaining use cases for .sequential() are<br>
    just modifiers to forEach to constrain it, in order, to the current<br>
    thread.<br>
<br>
    As in:<br>
       ints().parallel().filter(i -&gt; isPrime(i))<br>
             .sequential().forEach(System._<u></u>_out::println)<br>
<br>
    Which could be replaced by<br>
    .__<u></u>forEachSequentialAndOrderedInC<u></u>__urrentThread(), with a suitably<br>
    better name.  Which could further be simplified to ditch the &quot;in<br>
    current thread&quot; part by doing some locking in the implementation,<br>
    which brings us to .forEachOrdered(action).  Which nicely<br>
    complements .collectUnordered, and the two actually stand better<br>
    with their duals present (reduce is by default ordered; forEach is<br>
    by default unordered.)<br>
<br>
    The &quot;put it anywhere&quot; behavior of .parallel was completely<br>
    bootstrapped on the &quot;put it anywhere&quot; nature of .sequential; we<br>
    never really set out to support transitions in the API.<br>
<br>
    So, pulling the rug out from under the house of cards, I think we<br>
    can fall back to:<br>
<br>
    1.  Modify semantics of .sequential and .parallel to apply globally<br>
    to the entire pipeline.  This works because pipelines are fully lazy<br>
    anyway, so we don&#39;t commit to seq-ness/par-ness until we hit the<br>
    terminal op.  So they are functional versions of &quot;set the seq/par<br>
    bit in the source&quot;.  And that simplifies the specification of<br>
    seq/par down to a single property of the entire pipeline -- much<br>
    easier to spec.<br>
<br>
    2.  Add .forEachOrdered.  For sequential streams, this is just<br>
    .forEach.  For par streams, we use a lock to avoid concurrent<br>
    invocation of the lambda, and loosen up the current behavior from<br>
    &quot;full barrier&quot; to &quot;partial barrier&quot;, so that when the next chunk is<br>
    available, we can start working right away.  This is easy to<br>
    accomplish using the existing AbstractTask machinery.<br>
<br>
<br>
    Before we go there, does anyone have use cases for .sequential() /<br>
    .parallel() that *don&#39;t* put the parallel right after the source, or<br>
    the sequential right before a forEach?<br>
<br>
<br>
</blockquote>
</blockquote></div>