RFR: 8359919: Minor java.util.concurrent doc improvements
Viktor Klang
vklang at openjdk.org
Thu Jun 19 09:35:58 UTC 2025
On Wed, 18 Jun 2025 18:55:19 GMT, Pavel Rappo <prappo at openjdk.org> wrote:
>> This collects miscellaneous open issues that can be resolved with documentation updates; each indicated by adding JDK issue numbers
>
> src/java.base/share/classes/java/util/concurrent/Flow.java line 64:
>
>> 62: * control required in most implementations (for example {@link
>> 63: * SubmissionPublisher}), and omits some error processing needed to
>> 64: * fully conform to ReactiveStream rules.
>
> I had to re-read the comments on the relevant [issue] from 2018; thanks for fixing it.
>
> Nit: ReactiveStream. Earlier in that doc comment, it is referred to as "reactive-streams". We should choose one of those.
>
> We can say reactive-streams or Reactive Streams. The latter seems to be the term of choice at https://github.com/reactive-streams/reactive-streams-jvm/.
>
> @viktorklang-ora, thoughts?
>
> [issue]: https://bugs.openjdk.org/browse/JDK-8210149
@pavelrappo Indeed, "Reactive Streams" would be better. Perhaps something like this:
Suggestion:
* fully conform to the Reactive Streams specification.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25880#discussion_r2156574214
More information about the core-libs-dev
mailing list