ctubbsii commented on PR #2967: URL: https://github.com/apache/accumulo/pull/2967#issuecomment-1264095809
> I'm not sure I like the idea of overriding the cancel method. Handling the InterruptedException is the standard way of handling a canceled future so I don't think we should deviate from that pattern as that's the intended way for handling it. Okay, that's fine. I haven't looked into it very closely. I was just thinking it might be nice to have some degree of confidence on whether it succeeded or not upon cancellation, and we would know that if we interrupted between retries. -- 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]
