Github user parthchandra commented on a diff in the pull request:
https://github.com/apache/drill/pull/1144#discussion_r184242257
--- Diff: src/main/resources/checkstyle-config.xml ---
@@ -30,10 +30,15 @@
<module name="NeedBraces"/>
+ <module name="IllegalCatch">
--- End diff --
So what do you tell the user when you get a runtime exception (any
exception) that is the result of a bug? It is silly to show them a stack trace
that does not help them. It is better to let the user know that there was an
internal error and they should log a bug with support or look for help on the
dev list.
---
