[GitHub] drill pull request #1144: DRILL-6202: Deprecate usage of IndexOutOfBoundsExc...

[parthchandra]

Github user parthchandra commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1144#discussion_r183530901
  
    --- Diff: src/main/resources/checkstyle-config.xml ---
    @@ -30,10 +30,15 @@
     
         <module name="NeedBraces"/>
     
    +    <module name="IllegalCatch">
    --- End diff --
    
    I think IOBE should be caught and converted to a UserException so that the 
end user does not get an incomprehensible error message. 


---