chore(deps): upgrade java to 25, scylla-driver to 3.11.5.14 and add Renovate#35
Open
CodeLieutenant wants to merge 3 commits intomasterfrom
Open
chore(deps): upgrade java to 25, scylla-driver to 3.11.5.14 and add Renovate#35CodeLieutenant wants to merge 3 commits intomasterfrom
CodeLieutenant wants to merge 3 commits intomasterfrom
Conversation
…vate - Bump com.scylladb:scylla-driver-core 3.10.2-scylla-1 -> 3.11.5.14. 3.11.5.14 adds richer OperationTimedOutException diagnostics (pool contention vs server slowness, in-flight/pending counts) and pulls in the LWT replica-only routing + DefaultResultSetFuture completion fixes from the 3.11.5.x line. - Bump bundled netty override 4.1.112.Final -> 4.1.132.Final and promote the version to a netty.version property in scylla/pom.xml. - Add root renovate.json: enables maven, dockerfile, and github-actions managers; groups ScyllaDB drivers, AWS SDK v2, Netty, Testcontainers, Jackson, SLF4J, and Maven plugins; pins java-driver-core 4.x out (binding is on the 3.x API) and gates major bumps on scylla/AWS behind explicit review. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
We need to test it all around change of java version is always breaking things Before we have a clear longevity run in SCT we shouldn't merge |
Author
Yes of course, this is just the placeholder for it. And the priority is #34 |
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.
Summary
This PR bundles three related upgrades:
Java 25
pom.xml, all GitHub Actions workflows,Dockerfile(noweclipse-temurin:25-jdk/jre-noble), andscripts/package.shdynamodb/pom.xmlfrom0.8.11→0.8.13— required because 0.8.11 does not support Java 25 class files (major version 69) and would crash at instrumentation timeubuntu-latest→ubuntu-24.04for reproducible buildsScyllaDB driver upgrade
com.scylladb:scylla-driver-corefrom3.10.2-scylla-1(≈2 years old) to3.11.5.14. The new release adds detailedOperationTimedOutExceptiondiagnostics — pool contention vs server-side slowness, in-flight and pending borrow counts — directly useful for triagingError inserting keyfailures under heavy YCSB load. Also rolls in LWT replica-only routing +DefaultResultSetFuturecompletion fixes from the 3.11.5.x line.netty-handler/netty-transport/netty-codec) from4.1.112.Final→ **4.1.132.Final`` and promote to anetty.versionproperty inscylla/pom.xml` for a single source of truth.Renovate
renovate.jsonat repo root so the Renovate GitHub App can keep Maven, Dockerfile, and GitHub Actions deps fresh going forward — with sensible groupings (ScyllaDB drivers, AWS SDK v2, Netty, Testcontainers, Jackson, SLF4J, Maven plugins, GH Actions, Docker base images) and major-bump guardrails.eclipse-temurinandcom.scylladb:java-driver-coremajor bumps are disabled — Java 25 is now the pinned build target, and the 4.x driver line would require a full API rewrite.Why Java 25
Java 25 is the next LTS release (GA September 2025). Adopting it now:
Verification
mvn -pl scylla -am clean package -DskipTests→ BUILD SUCCESS on Java 25mvn -pl dynamodb -am test→ passes with JaCoCo 0.8.13mvn -pl scylla dependency:treeconfirmsscylla-driver-core:3.11.5.14andio.netty:*:4.1.132.Finalresolve cleanlyTest plan
bin/ycsb.sh load scylla -P workloads/workloada -threads 100 -p recordcount=18000000 …) and verify new timeout diagnostics surface inOperationTimedOutExceptionNotes for reviewer
org.testcontainers:testcontainers:2.0.3indynamodb/pom.xmllooks like a pre-existing typo — latest is1.21.x. Worth a follow-up PR.scylla-driver-coreare intentionally kept so the resolved netty version stays under our control.