[ 
https://issues.apache.org/jira/browse/DRILL-5523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16015621#comment-16015621
 ] 

ASF GitHub Bot commented on DRILL-5523:
---------------------------------------

GitHub user arina-ielchiieva opened a pull request:

    https://github.com/apache/drill/pull/842

    DRILL-5523: Revert if condition in UnionAllRecordBatch changed in DRI…

    …LL-5419
    
    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 initial state so users won't see the regression 
till DRILL-4824 is merged.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/arina-ielchiieva/drill DRILL-5523

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/842.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #842
    
----

----


> Revert if condition in UnionAllRecordBatch changed in DRILL-5419
> ----------------------------------------------------------------
>
>                 Key: DRILL-5523
>                 URL: https://issues.apache.org/jira/browse/DRILL-5523
>             Project: Apache Drill
>          Issue Type: Bug
>    Affects Versions: 1.11.0
>            Reporter: Arina Ielchiieva
>            Assignee: Arina Ielchiieva
>             Fix For: 1.11.0
>
>
> 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 to 
> 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.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to