Skip to content

Replace SimpleStrategy with NetworkTopologyStrategy across codebase#882

Closed
roydahan wants to merge 2 commits into
scylladb:masterfrom
roydahan:simple-strategy
Closed

Replace SimpleStrategy with NetworkTopologyStrategy across codebase#882
roydahan wants to merge 2 commits into
scylladb:masterfrom
roydahan:simple-strategy

Conversation

@roydahan
Copy link
Copy Markdown
Collaborator

ScyllaDB has dropped support for SimpleStrategy. Update all tests, examples, benchmarks, and management utilities to use NetworkTopologyStrategy instead, while preserving the SimpleStrategy class definition in the driver for backward compatibility with Cassandra clusters.

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • I have provided docstrings for the public items that I want to introduce.
  • I have adjusted the documentation in ./docs/source/.
  • I added appropriate Fixes: annotations to PR description.

@roydahan roydahan marked this pull request as draft May 11, 2026 12:48
roydahan added 2 commits May 11, 2026 17:01
ScyllaDB has dropped support for SimpleStrategy. Update all tests,
examples, benchmarks, and management utilities to use
NetworkTopologyStrategy instead, while preserving the SimpleStrategy
class definition in the driver for backward compatibility with
Cassandra clusters.
@roydahan
Copy link
Copy Markdown
Collaborator Author

I see there is a PR that just got merged for that #830

@roydahan roydahan closed this May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant