docs: remove stale Java 8/11 framing now that JDK 17 is the minimum - #3487
Merged
pjfanning merged 1 commit intoAug 30, 2026
Merged
Conversation
Motivation: Pekko 2.0.x requires JDK 17 as the minimum supported runtime (see release-notes/releases-2.0.md and project/JdkOptions.scala), a hard break from the 1.x line which supported Java 8+. Several doc pages and one reference.conf comment still carried Java 8/11-era framing that no longer applies to the current major version. Modification: - remoting-artery.md: reworded the Aeron `--add-opens` note and the Flight Recorder section, which claimed JFR is enabled by "detecting JDK 11" even though there is no such runtime detection in FlightRecorderLoader.scala - it is controlled purely by the `pekko.java-flight-recorder.enabled` config flag. - remote-security.md: dropped a TLS cipher caveat about Java 8 runtimes, which are no longer supported. - stream/reactive-streams-interop.md: removed Java-8-specific Reactive Streams artifact/JavaFlowSupport caveats. - persistence.md, typed/cluster-sharding.md, typed/distributed-data.md: reworded the LevelDB/LMDB `--add-opens` notes so they no longer frame JDK 17 as an optional/special case. - actor/src/main/resources/reference.conf: fixed the same stale "JDK 11 or later" framing in the java-flight-recorder comment, which is snip-included into docs/src/main/paradox/general/configuration-reference.md. Result: Doc pages and the flight-recorder config comment reflect the current JDK 17+ baseline instead of pre-2.0 Java 8/11 assumptions. Tests: - Not run - docs and comment-only changes. References: None - identified while auditing docs/src/main/paradox for content out of date relative to the current Pekko version and JDK support.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pekko 2.0.x requires JDK 17 as the minimum supported runtime (see release-notes/releases-2.0.md and project/JdkOptions.scala).
Modification:
remoting-artery.md: reworded the Aeron--add-opensnote and the Flight Recorder section, which claimed JFR is enabled by "detecting JDK 11" even though there is no such runtime detection inFlightRecorderLoader.scala- it is controlled purely by thepekko.java-flight-recorder.enabledconfig flag.remote-security.md: dropped a TLS cipher caveat about Java 8 runtimes, which are no longer supported.stream/reactive-streams-interop.md: removed Java-8-specific Reactive Streams artifact/JavaFlowSupport caveats.persistence.md,typed/cluster-sharding.md,typed/distributed-data.md: reworded the LevelDB/LMDB--add-opensnotes so they no longer frame JDK 17 as an optional/special case.actor/src/main/resources/reference.conf: fixed the same stale "JDK 11 or later" framing in the java-flight-recorder comment, which is snip-included into docs/src/main/paradox/general/configuration-reference.md.Result:
Doc pages and the flight-recorder config comment reflect the current JDK 17+ baseline instead of pre-2.0 Java 8/11 assumptions.