You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Ashutosh Chauhan <ha...@apache.org> on 2015/05/21 02:44:17 UTC

Review Request 34515: ReduceSinkDedup undoes Distinct Rewrite

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34515/
-----------------------------------------------------------

Review request for hive and John Pullokkaran.


Bugs: HIVE-10774
    https://issues.apache.org/jira/browse/HIVE-10774


Repository: hive-git


Description
-------

ReduceSinkDedup undoes Distinct Rewrite


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/Context.java a74bbbe 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveConfigContext.java 0e559e0 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/cost/HiveAlgorithmsConf.java 83454ea 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/cost/HiveAlgorithmsUtil.java 6840418 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveExpandDistinctAggregatesRule.java 73c7cac 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java 404b759 
  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 4760a22 
  ql/src/test/results/clientpositive/tez/limit_pushdown.q.out 2a41aae 
  ql/src/test/results/clientpositive/tez/mrr.q.out d90b27f 
  ql/src/test/results/clientpositive/tez/vectorization_limit.q.out 1c5b51f 

Diff: https://reviews.apache.org/r/34515/diff/


Testing
-------

Existing tests.


Thanks,

Ashutosh Chauhan