Re: RFR: 8272137: Make Collection and Optional classes streamable

2021-08-14 Thread CC007
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional I read through [these posts](http://mail.openjdk.java.net/pipermail/lambda-libs-spec-experts/2013-June/thread.html#1910) and while I did see good reaso

Re: RFR: 8272137: Make Collection and Optional classes streamable

2021-08-14 Thread Brian Goetz
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional To reiterate: These issues were explored in the JSR 335 EG and it was agreed that this abstraction did not carry its weight. In any case, it is prematur

Re: RFR: 8272137: Make Collection and Optional classes streamable

2021-08-14 Thread CC007
On Sun, 15 Aug 2021 01:07:10 GMT, Brian Goetz wrote: > I understand what you are proposing. I do not believe Streamable carries its > weight. That argument seems subjective. Could you elaborate on that in an objective manner? - PR: https://git.openjdk.java.net/jdk/pull/5050

Re: RFR: 8272137: Make Collection and Optional classes streamable

2021-08-14 Thread Brian Goetz
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional I understand what you are proposing. I do not believe Streamable carries its weight. Sent from my iPad > On Aug 14, 2021, at 8:53 PM, CC007 ***@**

Re: RFR: 8272137: Make Collection and Optional classes streamable

2021-08-14 Thread CC007
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional Changed the title, since it seemed to cause confusion. - PR: https://git.openjdk.java.net/jdk/pull/5050

Re: RFR: 8272137: Make Collection and Optional classes streamable

2021-08-14 Thread CC007
On Sat, 14 Aug 2021 15:13:39 GMT, Brian Goetz wrote: > I object to this change. These issues were explored in the JSR 335 EG and it > was agreed that this abstraction did not carry its weight. In any case, it is > premature to bring a PR for a significant API change that has not been > discuss

Re: RFR: 8272137: Make Iterable classes streamable

2021-08-14 Thread Brian Goetz
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional I object to this change. These issues were explored in the JSR 335 EG and it was agreed that this abstraction did not carry its weight. In any case, i

Re: RFR: 8272137: Make Iterable classes streamable

2021-08-14 Thread Brian Goetz
Starting with a PR is not the way to add significant API surface to the JDK. You have to first build consensus on the mailing list for the design concept before we start talking about code. FTR, Interfaces like these were already discussed and rejected in the JSR 335 design discussions.

Re: RFR: 8272137: Make Iterable classes streamable

2021-08-14 Thread CC007
On Sat, 14 Aug 2021 00:53:06 GMT, liach wrote: > Aren't all iterable implementations effectively streamable if they properly > implement `spliterator`? And the spliterator implementation can always be > sequential or parallel, dependent on how you feed it into `StreamSupport`. Yes indeed, but

Re: RFR: 8272137: Make Iterable classes streamable

2021-08-14 Thread CC007
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional The name of this pull request is a bit wrong, but it had to be the same as the JBS name. The interfaces don't make the Iterable have stream and parall

Re: RFR: 8272137: Make Iterable classes streamable

2021-08-14 Thread Rémi Forax
On Mon, 9 Aug 2021 12:28:23 GMT, CC007 wrote: > create Streamable and ParallelStreamable interface and use them in Collection > and Optional Hi Rick, I do not think that such interfaces are a good addition to the JDK, we do not want to promote the fact that you can create a Stream from an Ite