[GitHub] paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-04 Thread GitBox
paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#issuecomment-418506296 Note that is is, in general, very hard to test an exception such as these. We are catching programming errors.

[GitHub] paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-04 Thread GitBox
paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#issuecomment-418465629 @arina-ielchiieva, very good point. Having that information is very useful regardless of the source of error.

[GitHub] paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-04 Thread GitBox
paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#issuecomment-418433987 @KazydubB, thanks for the explanation. You are inserting code everywhere you believe an IOBE could be thrown.

[GitHub] paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-03 Thread GitBox
paul-rogers commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#issuecomment-418200233 I have a basic question on this one. Is an Index Out Of Bounds Exception (IOBE) an expected behavior for this