hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "pengcheng xiong" <pxi...@hortonworks.com>
Subject Re: Review Request 28797: Support Union Distinct
Date Mon, 29 Dec 2014 20:05:16 GMT

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

(Updated Dec. 29, 2014, 8:05 p.m.)


Review request for hive and John Pullokkaran.


Repository: hive-git


Description
-------

Current version (Hive 0.14) does not support union (or union distinct). It only supports union
all. In this patch, we try to add this new feature by rewriting union distinct to union all
followed by group by.


Diffs
-----

  itests/src/test/resources/testconfiguration.properties b0b5b2e 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/ASTConverter.java c02a65e

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/unionproc/UnionProcFactory.java a985c4f

  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 5d72e15 
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 4e98e9b 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java d9da15e 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SubQueryUtils.java 57868b7 
  ql/src/java/org/apache/hadoop/hive/ql/parse/UnparseTranslator.java 9ad6714 
  ql/src/test/queries/clientpositive/selectDistinctStar.q PRE-CREATION 
  ql/src/test/queries/clientpositive/unionDistinct_1.q PRE-CREATION 
  ql/src/test/queries/clientpositive/unionDistinct_2.q PRE-CREATION 
  ql/src/test/queries/clientpositive/union_remove_6.q bfce26d 
  ql/src/test/queries/clientpositive/union_remove_6_subq.q PRE-CREATION 
  ql/src/test/results/clientpositive/complex_alias.q.out 133ce91 
  ql/src/test/results/clientpositive/explain_logical.q.out 41c8a2b 
  ql/src/test/results/clientpositive/groupby_sort_1_23.q.out cb94faa 
  ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out bbc0de4 
  ql/src/test/results/clientpositive/input_part7.q.out 6094f9c 
  ql/src/test/results/clientpositive/join34.q.out e053e44 
  ql/src/test/results/clientpositive/join35.q.out 579920b 
  ql/src/test/results/clientpositive/multi_join_union.q.out 466f34b 
  ql/src/test/results/clientpositive/optimize_nullscan.q.out fb66f5e 
  ql/src/test/results/clientpositive/ppd_union_view.q.out 1729464 
  ql/src/test/results/clientpositive/ql_rewrite_gbtoidx.q.out 3630332 
  ql/src/test/results/clientpositive/selectDistinctStar.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/skewjoin_mapjoin5.q.out 0a4c26a 
  ql/src/test/results/clientpositive/skewjoin_mapjoin7.q.out aaf7de3 
  ql/src/test/results/clientpositive/skewjoinopt9.q.out 502376e 
  ql/src/test/results/clientpositive/tez/optimize_nullscan.q.out e598191 
  ql/src/test/results/clientpositive/tez/selectDistinctStar.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/tez/tez_smb_main.q.out b7c287d 
  ql/src/test/results/clientpositive/tez/unionDistinct_1.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/tez/unionDistinct_2.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/union22.q.out 672fa77 
  ql/src/test/results/clientpositive/union24.q.out 8755025 
  ql/src/test/results/clientpositive/union27.q.out f23f580 
  ql/src/test/results/clientpositive/union34.q.out 9ee382b 
  ql/src/test/results/clientpositive/unionDistinct_1.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/unionDistinct_2.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/union_ppr.q.out 0610337 
  ql/src/test/results/clientpositive/union_remove_12.q.out f9fd323 
  ql/src/test/results/clientpositive/union_remove_13.q.out b8913e2 
  ql/src/test/results/clientpositive/union_remove_14.q.out 28e4bb6 
  ql/src/test/results/clientpositive/union_remove_6_subq.q.out PRE-CREATION 

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


Testing
-------


Thanks,

pengcheng xiong


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message