bereng commented on PR #2657: URL: https://github.com/apache/cassandra/pull/2657#issuecomment-1704827402
The only thing that itches me is the complete removal of the 'fail' path. That is the removal of the legit timeouts whose reasons I understand thanks to the explanations on the ticket. I have no better proposal the the check `maxNodesDown()` already does. So LGTM +1 -- 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]

