----------------------------------------------------------- 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. Changes ------- rebase on HIVE-9133 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 (updated) ----- 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
