scylladb / scylla-code-samples

Code samples for working with ScyllaDB
Apache License 2.0
237 stars 130 forks source link

updated scylla version and removed simplestrategy #204

Closed guy9 closed 8 months ago

guy9 commented 1 year ago

@tzach please review

tarzanek commented 8 months ago

(so ideally network strategy change should be in separate commit, so you have isolated patch) why going for exact scylla version @guy9 ? I'd think we can just leave latest and if something will break people will report, it's probably better than people using old Scyllas or us changing version every few months, isn't it? @tzach ?

tzach commented 8 months ago

LGTM