<div dir="ltr">My point is that we can create Gatherers that gather indexed elements (indexed gatherer), and another factory wraps the Gatherer so the index boxing and automatically performed before sending to the indexed gatherer. All other stream operations, like indexed filter, indexed collect, etc. can all be done through the indexed gatherer.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Apr 20, 2024 at 7:13 PM ІП-24 Олександр Ротань <<a href="mailto:rotan.olexandr@gmail.com">rotan.olexandr@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">I am sorry, but I feel like I am missing the point of your response and how is it related to what I have said.<div dir="auto"><br></div><div dir="auto">Regarding wrapping and unwrapping indexed pairs, one of advantages of the approach I have suggested is that EnumeratedStream is still a stream and all index-unaware ops could still be applied to it</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 21, 2024, 03:08 - <<a href="mailto:liangchenblue@gmail.com" target="_blank">liangchenblue@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">We must convert index-processing operations to a `gather(Gatherers.scan(/* index gathering */))` immediate before the operation that uses the index, and immediately unwrap the indices afterwards.<div><div><br></div></div><div>Syntactically writing such 3 lines for every index-aware operation would be weird; I actually wonder if we can find a way to convert a Gatherer<Indexed<T>, ?, R> into a Gatherer<T, ?, R>; the nested gatherer itself will receive the enumerated values. I don't know if such a way to compose Gatherer has been studied, but this would allow us to define all indexed operations as gatherers instead of having to make new BiConsumers for indices. This would be interesting to look into. So the ideal form would be:</div><div><br>BiPredicate<Integer, T> filter = ...</div><div>Gatherer<Indexed<T>, ?, R> filterGatherer = Gatherer.of((_, indexed, sink) -> return (!filter.test(indexed.index(), indexed.value())) || sink.push(indexed.value()));<br>Gatherer<T, ?, R> wrapped = IndexedGatherer.wrap(filterGatherer); // a factory method performing wrapping</div><div>return stream.gather(wrapped) // and more</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Apr 20, 2024 at 6:47 PM ІП-24 Олександр Ротань <<a href="mailto:rotan.olexandr@gmail.com" rel="noreferrer" target="_blank">rotan.olexandr@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I would like to correct myself a bit: indexes should be assigned at the moment an element is received from upstream, not when it is passed to processing. Not sure if it will effectively change order of execution in actual implementation, but to put it this way would be more precise</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">вс, 21 апр. 2024 г. в 02:33, ІП-24 Олександр Ротань <<a href="mailto:rotan.olexandr@gmail.com" rel="noreferrer" target="_blank">rotan.olexandr@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hello again.<div><br></div><div>I have imagined implementation of enumerated streams in a slightly different way. I think enumerated streams should be a separate kind of stream like parallel, sequential or some primitive stream. This way enumeration could be deferred to the time that index is being consumed, This would eliminate issue with stream merging and also kind of smoothen the issue with parallel streams, as the indexed will follow the order elements have been passed to processing. </div><div><br></div><div>This could potentially lead to another issue with consistency of indexes through stream pipeline, but there could be a workaround like IdentityHashMaps that is received from previous enumerated streams.</div><div><br></div><div>This way we could make streams more efficient and also introduce a more user-friendly API, which is essentially a win-win situation.</div><div><br></div><div>Regarding concatenation specifically, we could either introduce separate methods for merging enumerated streams, where we define how conflicts are resolved, or just treat them as usual streams, so the result of concatenation of enumerated streams will result in a normal stream that should be enumerated once again. Also i don't really see how gathering elements into index-value pairs solves issue with conflicting indexes</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">вс, 21 апр. 2024 г. в 02:06, - <<a href="mailto:liangchenblue@gmail.com" rel="noreferrer" target="_blank">liangchenblue@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Oleksandr,<div>I fear that enumeration might not be applicable to all streams, especially parallel ones. If we have a parallel stream, we might not always process all elements in order, and even index generation can be unreliable. In addition, stream merging will become a headache. I think Gatherers.scan(() -> new Indexed<>(0, dummy), (indexed, value) -> new Indexed<>(indexed.index() + 1, value)) can create a Stream<Indexed<V>> which should serve your purpose.</div><div><br></div><div>And for expanded operations for enumerated streams, there are similar features for primitive streams already, where they have extra methods like summaryStatistics() compared to object streams. We most likely will have Gatherers that explicitly work on Stream<Indexed<V>> to support type-specific operations, partly due to Java's type limits and that Stream hierarchy is effectively closed.</div><div><br></div><div>Best,</div><div>Chen Liang</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Apr 20, 2024 at 4:07 PM ІП-24 Олександр Ротань <<a href="mailto:rotan.olexandr@gmail.com" rel="noreferrer" target="_blank">rotan.olexandr@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">My proposal regarding findIndex brought up a topic that, as I see, has been brought up here multiple times.<div><br></div><div>My idea is to extend the existing stream interface in the new EnumeratedStream interface. Such an approach has a number of advantages.</div><div><br></div><div>1. Some methods will be able to explicitly specify that they require an enumerated stream.</div><div><br></div><div>2. This would allow us to use an enumerated stream just as the default value stream, if the index is redundant at a certain processing step.</div><div><br></div><div>3. This could help introduce a more user-friendly interface: instead of passing pairs of index and value, they could be passed as separate variables, which will make code more concise.</div><div><br></div><div>Consider following example:</div><div>List.of(1, 2, 3).stream()<br>                .enumerated()<br>                .map(idx, value -> idx % 2 == 0 ? value : -value);<br></div><div><br></div><div>looks much better than </div><div>List.of(1, 2, 3).stream()<br>                .enumerated()<br>                .map(pair -> pair.idx % 2 == 0 ? pair.value : -pair.value);<br></div><div><br></div><div>However, I see some caveats with this approach, which relate to parallel streams:</div><div>when a stream is backed by a collection, you might expect assigned indexes to represent order of iteration through collection, especially when talking about sequential collections. Assigning indexes in such streams could be a difficult task in a parallel environment. It should either assign index to a stream elements at the moment when stream becomes parallelized, which is also impossible if already parallelized stream is being enumerated, and also wont work for infinite streams, or make enumerated stream at least partially sequential by passing elements to processing in order they were passed to stream at the first place, which is also hard or maybe even impossible to achieve.</div><div><br></div><div>Also, side note: methods that already accept 2 parameters might become kinda verbose, like reduce that will have to accept for params (idx1, val1, idx2, val2), but i think that is not a big deal</div><div><br></div><div><br></div></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>