Jesus Camacho Rodriguez commented on HIVE-14957:

[~pxiong], if I remember correctly, the reason we match the parent op is that 
the rule should not be triggered at the top of the plan (see the header comment 
of the rule). Let's wait for QA results, but if this is the case, it would 
suffice with adding the other children when we copy the parent operator.

> HiveSortLimitPullUpConstantsRule misses branches when parent operator is Union
> ------------------------------------------------------------------------------
>                 Key: HIVE-14957
>                 URL: https://issues.apache.org/jira/browse/HIVE-14957
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Pengcheng Xiong
>            Assignee: Pengcheng Xiong
>         Attachments: HIVE-14957.01.patch
> {code}
> call.transformTo(parent.copy(parent.getTraitSet(), 
> ImmutableList.of(relBuilder.build())));
> {code}
> When parent is an union operator which has 2 inputs, the parent.copy will 
> only copy the one that has SortLimit and ignore the other branches.

This message was sent by Atlassian JIRA

Reply via email to