beobal commented on PR #3424: URL: https://github.com/apache/cassandra/pull/3424#issuecomment-2231444026
> +1, LTGM. The only question is whether or not it is worth to check other spots where we are waiting uninterruptibly, in places such as `PaxosBackedProcessor` and `PeerLogFetcher`, etc? I think the instances in `PaxosBackedProcessor` are ok as they already specify timeouts. I don't think there are any uninterruptible waits in `PeerLogFetcher`, but there were a couple in `RemoteProcessor` which I've made interruptible. These were already wrapped by catches for `InterruptedException` so that seems reasonable to me. I've attached a new CI summary to the jira, which again shows no new failures. -- 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]
