serban21 opened a new pull request, #106: URL: https://github.com/apache/cassandra-analytics/pull/106
This includes some changes from #93 , so it's better to merge it only after the other one is merged. I used `tokenPartitioner.ring().instances()` instead of making a new request to get ring information since the tokenPartitioner is built just before this. An argument can be made that this is not the case for clearing snapshots, since that is happening (much) later. On the other hand if the ring had instances added/removed/replaced in between the new instances would not have the snapshots anyway, and the old instances will be removed. -- 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]
