You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Jesús Camacho Rodríguez <jc...@hortonworks.com> on 2015/08/03 14:21:25 UTC
Re: Review Request 36877: CBO: Calcite Operator To Hive Operator
(Calcite Return Path) : fix reduce_deduplicate optimization
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36877/#review93895
-----------------------------------------------------------
Ship it!
Ship It!
- Jesús Camacho Rodríguez
On July 29, 2015, 10:40 p.m., pengcheng xiong wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36877/
> -----------------------------------------------------------
>
> (Updated July 29, 2015, 10:40 p.m.)
>
>
> Review request for hive and Jesús Camacho Rodríguez.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> The patch creates new plan which is more efficient than the previous one in master.
>
>
> Diffs
> -----
>
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/AbstractCorrelationProcCtx.java 174685b
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/CorrelationUtilities.java 64bef21
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java 7b5f9b2
> ql/src/test/results/clientpositive/multi_insert_move_tasks_share_dependencies.q.out 8f9dd12
> ql/src/test/results/clientpositive/ptf.q.out e61703c
> ql/src/test/results/clientpositive/spark/multi_insert_move_tasks_share_dependencies.q.out cddf923
> ql/src/test/results/clientpositive/spark/ptf.q.out 647b83e
> ql/src/test/results/clientpositive/spark/union_remove_6_subq.q.out b5a2a75
> ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out 6960bee
> ql/src/test/results/clientpositive/tez/explainuser_1.q.out 1c49f52
> ql/src/test/results/clientpositive/tez/ptf.q.out 88d1a98
> ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out c2e9b1a
> ql/src/test/results/clientpositive/union_remove_6_subq.q.out a38548f
> ql/src/test/results/clientpositive/vectorized_ptf.q.out 79edb0e
>
> Diff: https://reviews.apache.org/r/36877/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> pengcheng xiong
>
>