Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1144#discussion_r179231151
--- Diff: src/main/resources/checkstyle-config.xml ---
@@ -30,10 +30,15 @@
<module name="NeedBraces"/>
+ <module name="IllegalCatch">
--- End diff --
I commented out the default rule value with the addition of
IndexOutOfBoundsException. IndexOutOfBoundsException can be used, but it is not
common to handle it.---
