You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/08/08 16:05:42 UTC

[GitHub] [accumulo] ctubbsii commented on issue #2852: ZooZap catches overly broad exceptions

ctubbsii commented on issue #2852:
URL: https://github.com/apache/accumulo/issues/2852#issuecomment-1208318075

   The comment that spawned this issue was very specific to the code where it was newly being introduced. We already have occurrences where we catch overly broad exceptions when we shouldn't, or rewrap RTEs with another RTE as a result, as well as occurrences where we throw overly generic exception types. The point of the original comment was to avoid introducing new such occurrences, not to address all the existing ones. You'd have to go back to the original comment/issue to address the specific ones being added here that should be fixed.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org