Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1144#discussion_r183571786
--- Diff: src/main/resources/checkstyle-config.xml ---
@@ -30,10 +30,15 @@
<module name="NeedBraces"/>
+ <module name="IllegalCatch">
--- End diff --
IMO the same applies to all run-time exceptions (NPE, CCE, IOBE) and likely
conversion for NPE is the same as for IOBE. Note that most likely the root
cause of an IOBE or NPE is a bug and not an end user error, misconfiguration,
invalid input or a lack of resources, so end user won't be able to "fix" NPE or
IOBE.
---
