Remove unused DSE Graph support - #1036
Conversation
|
Important Review skippedToo many files! This PR contains 202 files, which is 102 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 (202)
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 |
b584b1d to
f9afd1d
Compare
f9afd1d to
aa660cf
Compare
Closes #1028. Part of #1025.
What and why
Remove DSE Graph APIs, request processors, GraphSON/GraphBinary code, metrics, tests, OSGi fixtures, and documentation. ScyllaDB does not support or test DSE Graph, so deleting it removes 24k lines of unverified production and test code from the maintenance surface.
The TinkerPop, TinkerGraph, Gremlin processor, and JavaTuples dependency chain is removed. DSE geometry and schema metadata remain for their dedicated follow-up issues.
Validation
mvn -pl core,metrics/micrometer,metrics/microprofile,test-infra test— 3,724 tests, no failures or errors.mvn -pl integration-tests,osgi-tests -am -DskipTests test-compilemvn -DskipTests install— full 17-module reactor, including shaded and distribution packaging.Dependencies: none. Cherry-picked commits: none.