dlmarion commented on PR #2946: URL: https://github.com/apache/accumulo/pull/2946#issuecomment-1255147525
> [This](https://docs.oracle.com/javase/tutorial/essential/exceptions/runtime.html) is a nice clarification from Oracle docs: "If a client can reasonably be expected to recover from an exception, make it a checked exception. If a client cannot do anything to recover from the exception, make it an unchecked exception." Do you think this should not be merged then and leave CryptoException as a RuntimeException. I was thinking that having a known checked Exception in the SPI methods makes sense so that errors specific to crypto can be handled. -- 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]
