----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57944/#review170280 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java Lines 224 (patched) <https://reviews.apache.org/r/57944/#comment243083> Can you elaborate a bit about this change? Maybe a comment could be left on why it is needed. Further, since in L197 we check whether number of keys in child is less than number of keys in parent, and here we enforce greater than... is it not the same as changing condition in L197 to _cRSc.getKeyCols().size() != pRSNc.getKeyCols().size()_? - Jesús Camacho Rodríguez On March 27, 2017, 7:54 a.m., Remus Rusanu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57944/ > ----------------------------------------------------------- > > (Updated March 27, 2017, 7:54 a.m.) > > > Review request for hive, Ashutosh Chauhan and Jesús Camacho Rodríguez. > > > Repository: hive-git > > > Description > ------- > > HIVE-15708 Upgrade Calcite to 1.12 and Avatica to 1.9 > > > Diffs > ----- > > druid-handler/pom.xml 926c53b > > druid-handler/src/java/org/apache/hadoop/hive/druid/io/DruidQueryBasedInputFormat.java > 0b35428 > pom.xml 4c73e27 > ql/pom.xml 7db0ede > > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMaterializedViewsRegistry.java > 1d78b4c > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/CalciteSemanticException.java > 0c6996c > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveCalciteUtil.java > 6ccd879 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HivePlannerContext.java > 9a65de3 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveRelShuttleImpl.java > 2aadf50 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveExtractDate.java > 4edc4df > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveFilterJoinRule.java > 4b8568e > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/views/HiveMaterializedViewFilterScanRule.java > 38d7906 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/stats/HiveRelMdPredicates.java > 69e157e > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/ASTBuilder.java > 0dc0c24 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/ASTConverter.java > 27990a2 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/ExprNodeConverter.java > e840938 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/RexNodeConverter.java > a05b89c > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java > 2b075be > ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java e75e892 > > ql/src/test/org/apache/hadoop/hive/ql/optimizer/calcite/TestCBORuleFiredOnlyOnce.java > 7229cc7 > ql/src/test/results/clientnegative/subquery_scalar_multi_rows.q.out 92f1365 > ql/src/test/results/clientpositive/cbo_rp_auto_join1.q.out fe4bc4f > ql/src/test/results/clientpositive/cbo_rp_outer_join_ppr.q.out 0bb18cc > ql/src/test/results/clientpositive/constprog2.q.out 7bfd0cf > ql/src/test/results/clientpositive/druid_basic2.q.out bc9410b > ql/src/test/results/clientpositive/druid_intervals.q.out ca3febf > ql/src/test/results/clientpositive/druid_timeseries.q.out 6b2ffe9 > ql/src/test/results/clientpositive/druid_topn.q.out 57d6586 > ql/src/test/results/clientpositive/filter_cond_pushdown.q.out 8c6f0f1 > ql/src/test/results/clientpositive/fouter_join_ppr.q.out 1897aa7 > ql/src/test/results/clientpositive/index_auto_unused.q.out 8a270d6 > ql/src/test/results/clientpositive/join45.q.out 249fe9c > ql/src/test/results/clientpositive/join_alt_syntax.q.out cef5272 > ql/src/test/results/clientpositive/join_cond_pushdown_1.q.out 9461a1b > ql/src/test/results/clientpositive/join_cond_pushdown_3.q.out 2205c4e > ql/src/test/results/clientpositive/join_cond_pushdown_unqual1.q.out 966330d > ql/src/test/results/clientpositive/join_cond_pushdown_unqual3.q.out 54a2d06 > ql/src/test/results/clientpositive/join_merging.q.out 79a7023 > ql/src/test/results/clientpositive/llap/auto_smb_mapjoin_14.q.out a867bd2 > ql/src/test/results/clientpositive/llap/auto_sortmerge_join_9.q.out c08a534 > ql/src/test/results/clientpositive/llap/constprog_semijoin.q.out 98d2328 > ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction_2.q.out > d291e7d > ql/src/test/results/clientpositive/llap/explainuser_1.q.out e806ef3 > ql/src/test/results/clientpositive/llap/explainuser_4.q.out 9804569 > ql/src/test/results/clientpositive/llap/lineage3.q.out 2f53e60 > ql/src/test/results/clientpositive/llap/mergejoin.q.out 64929cc > ql/src/test/results/clientpositive/llap/orc_ppd_decimal.q.out b5e8658 > ql/src/test/results/clientpositive/llap/orc_predicate_pushdown.q.out > 4bba265 > ql/src/test/results/clientpositive/llap/parquet_predicate_pushdown.q.out > 01348a6 > ql/src/test/results/clientpositive/llap/reduce_deduplicate_extended.q.out > 08a9bd1 > ql/src/test/results/clientpositive/llap/smb_mapjoin_14.q.out ddb7624 > ql/src/test/results/clientpositive/llap/subquery_exists.q.out 9e0a1d7 > ql/src/test/results/clientpositive/llap/subquery_in.q.out 1cbbe8f > ql/src/test/results/clientpositive/llap/subquery_multi.q.out d2016b3 > ql/src/test/results/clientpositive/llap/subquery_notin.q.out f1efb64 > ql/src/test/results/clientpositive/llap/subquery_scalar.q.out f6dc397 > ql/src/test/results/clientpositive/llap/subquery_select.q.out 6fb253b > ql/src/test/results/clientpositive/llap/vector_auto_smb_mapjoin_14.q.out > 10bd85e > ql/src/test/results/clientpositive/llap/vector_between_columns.q.out > 923e579 > ql/src/test/results/clientpositive/llap/vector_interval_mapjoin.q.out > eee37d0 > ql/src/test/results/clientpositive/llap/vectorization_0.q.out d919035 > ql/src/test/results/clientpositive/louter_join_ppr.q.out 2f6cdfd > ql/src/test/results/clientpositive/mapjoin1.q.out e103eff > ql/src/test/results/clientpositive/mergejoin.q.out 0ceb0e3 > ql/src/test/results/clientpositive/mergejoins.q.out 1023f61 > ql/src/test/results/clientpositive/outer_join_ppr.q.out 2bf6d61 > ql/src/test/results/clientpositive/perf/query1.q.out 53acdcd > ql/src/test/results/clientpositive/perf/query13.q.out 941a3a1 > ql/src/test/results/clientpositive/perf/query14.q.out 051d837 > ql/src/test/results/clientpositive/perf/query15.q.out 4f4dcc5 > ql/src/test/results/clientpositive/perf/query16.q.out dceb3a5 > ql/src/test/results/clientpositive/perf/query19.q.out 0ddcd83 > ql/src/test/results/clientpositive/perf/query21.q.out 67a49e2 > ql/src/test/results/clientpositive/perf/query23.q.out 85cee23 > ql/src/test/results/clientpositive/perf/query30.q.out d59098b > ql/src/test/results/clientpositive/perf/query31.q.out 433d419 > ql/src/test/results/clientpositive/perf/query32.q.out 6311054 > ql/src/test/results/clientpositive/perf/query37.q.out 1b61198 > ql/src/test/results/clientpositive/perf/query40.q.out a45fdaf > ql/src/test/results/clientpositive/perf/query48.q.out 3b390f2 > ql/src/test/results/clientpositive/perf/query5.q.out d3ec922 > ql/src/test/results/clientpositive/perf/query58.q.out f7e6a08 > ql/src/test/results/clientpositive/perf/query6.q.out 06cd19f > ql/src/test/results/clientpositive/perf/query64.q.out 468c1bc > ql/src/test/results/clientpositive/perf/query65.q.out 060cddf > ql/src/test/results/clientpositive/perf/query69.q.out 90224ce > ql/src/test/results/clientpositive/perf/query70.q.out bf90cdd > ql/src/test/results/clientpositive/perf/query72.q.out 828aa28 > ql/src/test/results/clientpositive/perf/query75.q.out f78ce8a > ql/src/test/results/clientpositive/perf/query80.q.out 7338d9c > ql/src/test/results/clientpositive/perf/query81.q.out 8234780 > ql/src/test/results/clientpositive/perf/query82.q.out 9c0042c > ql/src/test/results/clientpositive/perf/query85.q.out e7f7348 > ql/src/test/results/clientpositive/perf/query98.q.out 1bae9be > ql/src/test/results/clientpositive/pointlookup2.q.out 3438c74 > ql/src/test/results/clientpositive/pointlookup3.q.out 2c3e39f > ql/src/test/results/clientpositive/ppd_join2.q.out d5f5016 > ql/src/test/results/clientpositive/ppd_join3.q.out 3a8c1d7 > ql/src/test/results/clientpositive/ppd_outer_join1.q.out 8ec267c > ql/src/test/results/clientpositive/reduce_deduplicate_extended2.q.out > 8e35d1b > ql/src/test/results/clientpositive/router_join_ppr.q.out cc2b07e > ql/src/test/results/clientpositive/semijoin5.q.out fd8e372 > ql/src/test/results/clientpositive/spark/auto_smb_mapjoin_14.q.out 3b0d96c > ql/src/test/results/clientpositive/spark/auto_sortmerge_join_9.q.out > 2b09fcf > ql/src/test/results/clientpositive/spark/constprog_semijoin.q.out 89c2735 > ql/src/test/results/clientpositive/spark/join_alt_syntax.q.out 7a9baca > ql/src/test/results/clientpositive/spark/join_cond_pushdown_1.q.out 11d235b > ql/src/test/results/clientpositive/spark/join_cond_pushdown_3.q.out a53db90 > ql/src/test/results/clientpositive/spark/join_cond_pushdown_unqual1.q.out > 949d442 > ql/src/test/results/clientpositive/spark/join_cond_pushdown_unqual3.q.out > b5c91b6 > ql/src/test/results/clientpositive/spark/join_merging.q.out fcf80bd > ql/src/test/results/clientpositive/spark/louter_join_ppr.q.out 2e1c74d > ql/src/test/results/clientpositive/spark/mapjoin1.q.out aa96fc5 > ql/src/test/results/clientpositive/spark/mergejoins.q.out a3f8f8d > ql/src/test/results/clientpositive/spark/outer_join_ppr.q.out 2d0270b > ql/src/test/results/clientpositive/spark/ppd_join2.q.out 8d97034 > ql/src/test/results/clientpositive/spark/ppd_join3.q.out 99ded53 > ql/src/test/results/clientpositive/spark/ppd_outer_join1.q.out 1fc8232 > ql/src/test/results/clientpositive/spark/router_join_ppr.q.out 771609a > ql/src/test/results/clientpositive/spark/smb_mapjoin_14.q.out b5cef85 > ql/src/test/results/clientpositive/spark/subquery_exists.q.out 1901dba > ql/src/test/results/clientpositive/spark/subquery_in.q.out 7f53d27 > ql/src/test/results/clientpositive/spark/vectorization_0.q.out d8801f4 > ql/src/test/results/clientpositive/subq_where_serialization.q.out f689651 > ql/src/test/results/clientpositive/subquery_exists.q.out b271529 > ql/src/test/results/clientpositive/subquery_in_having.q.out 7473928 > ql/src/test/results/clientpositive/subquery_notin_having.q.out 375a48a > ql/src/test/results/clientpositive/subquery_unqualcolumnrefs.q.out 94210fd > ql/src/test/results/clientpositive/udf_between.q.out 2bb7264 > ql/src/test/results/clientpositive/vector_between_columns.q.out 8a79a51 > ql/src/test/results/clientpositive/vector_interval_mapjoin.q.out d2a879d > ql/src/test/results/clientpositive/vector_non_constant_in_expr.q.out > d9591d0 > ql/src/test/results/clientpositive/vector_string_decimal.q.out 3540635 > > > Diff: https://reviews.apache.org/r/57944/diff/1/ > > > Testing > ------- > > > Thanks, > > Remus Rusanu > >
