dcapwell commented on PR #3694: URL: https://github.com/apache/cassandra/pull/3694#issuecomment-2515826347
Here is a patch to fix the empty range issue https://github.com/dcapwell/cassandra/tree/feedback/pr-3694 To dump the history here: 1) single node JVM Dtest cluster had `Long.MAX_VALUE` for the token which caused TCM to produce an "empty" range (containing 0 partitions).. by decoupling Accord from TCM ranges we can merge touching ranges and avoid this issue. 2) TokenRange overrides `someIntersectingRoutingKey` to avoid using `+Inf` as the home key, but this is a valid key so removed... by fixing the above this will now match the what is present in the stores (the empty range got dropped so we didn't have +Inf in stores, only shards) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
