[DRILL-5523] Revert if condition in UnionAllRecordBatch changed in DRILL-5419

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.11.0
• Fix Version/s: 1.11.0
• Labels: ready-to-commit
• Assignee: Arina Ielchiieva
• Reporter: Arina Ielchiieva
• Reviewer: Vova Vysotskyi
• Created: 18/May/17 10:49
• Updated: 22/May/17 08:50
• Resolved: 22/May/17 08:50

Description

If condition in UnionAllRecordBatch which was changed to correct one in DRILL-5419 but this change caused the regression (DRILL-5521).
The root cause is not in change itself but rather in missing indication of child column in map types when it is null. There is a big project upcoming re-working json reader implementation, including map types (DRILL-4824) which will fix the problem.

So far reverting change to previous one so users won't see the regression till DRILL-4824 is merged.

Issue Links

• duplicates: DRILL-5521 IndexOutOfBoundsException during union all for map types
• duplicates: DRILL-4824 Null maps / lists and non-provided state support for JSON fields. Numeric types promotion.
• links to: GitHub Pull Request #842