keith-turner commented on PR #3396: URL: https://github.com/apache/accumulo/pull/3396#issuecomment-1544576911
> I think this conflicts with the advice given https://github.com/apache/accumulo/pull/3375#issuecomment-1536311772. A wrapped error may not always come frrom Accumulo code or user plugin code, it could come from java code or third party libraries. For example the java future class may wrap an Error in an ExecutionException then something in the Accumulo code may wrap the ExecutionException with a RuntimeException, an error occured its just not at the top level. We don't have to make this change, but if the intention is to halt on Error I think its reasonable to expect it to be possibly wrapped for many many different reasons. -- 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]
