Remove unused DSE protocol and continuous paging support - #1041
Remove unused DSE protocol and continuous paging support#1041dkropachev wants to merge 2 commits into
Conversation
|
Important Review skippedToo many files! This PR contains 253 files, which is 153 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. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Pro Plus Run ID: 📒 Files selected for processing (253)
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 |
6e93ec7 to
59b3f78
Compare
Closes #1032
Why
ScyllaDB does not support or test DSE continuous paging or DSE protocol V1/V2. Deleting this code reduces production surface area and the test/support burden while retaining ordinary reactive CQL execution.
What
Dependency and rebase
This PR depends on #1036 and includes its exact current Graph-removal commit
aa660cfc5a. The protocol-removal commit is59b3f78b6b. After #1036 merges, rebase ontoscylla-4.x, dropaa660cfc5a, and retain/reapply59b3f78b6b.Verification
mvn -pl core clean test— 3,418 unit tests plus 76 Reactive Streams TCK tests passmvn -pl metrics/micrometer,metrics/microprofile testmvn -pl integration-tests -am -DskipTests test-compilemvn -DskipTests install— all 17 modules pass