[GitHub] vrozov commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support

2018-07-21 Thread GitBox
vrozov commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support URL: https://github.com/apache/drill/pull/1244#issuecomment-406811356 @paul-rogers The change to `NullableVarCharVector` that was part of your PR exposed a bug that is part of the existing

[GitHub] vrozov commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support

2018-07-20 Thread GitBox
vrozov commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support URL: https://github.com/apache/drill/pull/1244#issuecomment-406668760 @paul-rogers I disagree that the bug was introduced in the prior changes to this PR. The bug is already part of

[GitHub] vrozov commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support

2018-07-19 Thread GitBox
vrozov commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support URL: https://github.com/apache/drill/pull/1244#issuecomment-406364011 @paul-rogers Did you try to pull changes from PR #1383?