RFR: 8319123: Implement JEP 461: Stream Gatherers (Preview) [v14]
Rob Spoor
duke at openjdk.org
Wed Jun 12 17:34:30 UTC 2024
On Thu, 30 Nov 2023 14:34:32 GMT, Viktor Klang <vklang at openjdk.org> wrote:
>> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461)
>
> Viktor Klang has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 32 commits:
>
> - Lowering upper boundary of gatherer composition testing to 512
> - Javadoc clarifications for Gatherer type parameters,
> correcting descriptions of Integrator type parameters,
> and more consistently use "initializer function" instead
> of "supplier function".
> - Addressing trailing whitespace complaint
> - Adding @implSpec for Gatherer.andThen, plus adds tests to verify the default impls for Gatherer-related interfaces
> - Clarifying that Stream#gather is a stateful intermediate operation
> - Improvements after feedback
> - Unpacking the rethrown exceptions from Gatherers.mapConcurrent()
> - Addressing last review feedback
> - Apply suggestions from code review
>
> Integrates Paul Sandoz's feedback/changes
>
> Co-authored-by: Paul Sandoz <paul.d.sandoz at googlemail.com>
> - Minor improvements to Gatherer Javadoc
> - ... and 22 more: https://git.openjdk.org/jdk/compare/69413690...49c94c7f
src/java.base/share/classes/java/util/stream/Gatherer.java line 296:
> 294: * @implSpec This method always returns the same instance.
> 295: *
> 296: * @see Gatherer#finisher()
Today I noticed that this is the wrong link. It should be `Gatherer#combiner()`, not `Gatherer#finisher()`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16420#discussion_r1636853721
More information about the core-libs-dev
mailing list