Remove residual DSE compatibility and build wiring - #1042
Open
dkropachev wants to merge 1 commit into
Open
Conversation
|
Important Review skippedToo many files! This PR contains 555 files, which is 455 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (555)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
dkropachev
force-pushed
the
cleanup/1033-remove-residual-dse
branch
3 times, most recently
from
August 31, 2026 16:44
0998e21 to
04dbf4e
Compare
nikagra
reviewed
Sep 1, 2026
nikagra
left a comment
nikagra
left a comment
There was a problem hiding this comment.
Review of the DSE removal. Two blockers inline — an inverted HCD guard in the CCM customizer and a serialized-field rename in Version — plus API-compatibility, revapi and changelog notes.
| return ""; // will never be used | ||
| } | ||
| }; | ||
| protected static class PlainTextAuthenticator implements SyncAuthenticator { |
There was a problem hiding this comment.
[Major] 🟠 Drops BaseDseAuthenticator, the deprecated PlainTextAuthenticator(Credentials), 3-arg Credentials and getAuthorizationId() with no deprecation cycle — and manual/core/authentication still points custom providers here.
| }, | ||
| { | ||
| "code": "java.field.removed", | ||
| "old": "field com.datastax.oss.driver.api.testinfra.ccm.CcmBridge.DSE_ENABLEMENT", |
There was a problem hiding this comment.
[Major] 🟠 DSE_ENABLEMENT is removed by this PR, so this ignore is needed now, not less. Meanwhile dsetool, reloadCore, the V* constants and the dropped SessionRule members get none, while core/revapi.json keeps dead TinkerPop rules.
| @@ -19,7 +19,6 @@ | |||
|
|
|||
| public enum BackendType { | |||
| CASSANDRA("Apache Cassandra"), | |||
| DSE("DSE"), | |||
| HCD("HCD"), | |||
There was a problem hiding this comment.
[Question] 🔵 DSE goes but HCD, its DataStax successor, stays — along with the version mapping, the DescribeIT/hcd fixtures and the DirectCompressionIT branch. Intentional, or a later slice? No CI lane exercises it.
dkropachev
force-pushed
the
cleanup/1033-remove-residual-dse
branch
from
September 1, 2026 23:31
04dbf4e to
88acdf4
Compare
dkropachev
force-pushed
the
cleanup/1033-remove-residual-dse
branch
from
September 2, 2026 15:27
88acdf4 to
98dcf8f
Compare
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
2 participants
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.
Closes #1033
Why
After the focused feature removals, deprecated DSE aliases and DSE-only test/build paths would still add untested production and maintenance surface. Removing them completes the cleanup while preserving the generic reactive API supported with Cassandra and Scylla.
What
DefaultDriverOptioncom.datastax.dseproduction packages are generic reactive CQL/mapper codeDependency and rebase
This integration PR depends on all preceding child PRs. Exact current source commits and their rebased commits on this branch:
a0fb0bdba8->72f50a3870a9b72c6695->8f2a762359aa660cfc5a->b5f12f5d8a0c3ca6bd84->0a852fe0201817ec3c50->b3d95e9b27697dc02e00->f8a3441fad59b3f78b6b->ff57760410The actual #1033 cleanup commit is
88acdf4ac7. After the prerequisite PRs merge, rebase ontoscylla-4.x, drop the seven prerequisite commits above, and retain/reapply88acdf4ac7.Verification
mvn -pl core test— 3,160 unit tests plus 76 Reactive Streams TCK tests passmvn clean -DskipTests install— all 17 modules pass, including OSGi, shaded, distributions, Revapi, and examplesmake compile-allpasses, including API leak checkscom.datastax.dseare the documented generic reactive CQL/mapper packages and their queue helper