dlmarion commented on PR #6019: URL: https://github.com/apache/accumulo/pull/6019#issuecomment-3656481600
> The way the compactor communicates cancellation is hidden and its brittle, makes it easier for someone to accidentally break these comms on the client or server side w/o realizing it. We could make communicating this more visible with a new RPC specifically for compaction cancelled. Or we could add a thrift enum to the existing failure RPC to specify the reason for the failure like ERROR or CANCELLATION. Good idea, I'll look at adding a Thrift enum. -- 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]
