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

Review request for hive.


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


Repository: hive-git


Description
-------

Add AggregateProjectMergeRule


Diffs
-----

  
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveAggregateProjectMergeRule.java
 PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java f26d1df 
  ql/src/test/results/clientpositive/alter_partition_coltype.q.out 9fc3c8d 
  ql/src/test/results/clientpositive/annotate_stats_groupby.q.out 82cc0da 
  ql/src/test/results/clientpositive/annotate_stats_groupby2.q.out 2cb1e84 
  ql/src/test/results/clientpositive/auto_join18.q.out 6dc7a63 
  ql/src/test/results/clientpositive/auto_join18_multi_distinct.q.out cc17ad1 
  ql/src/test/results/clientpositive/auto_join27.q.out 16a7f02 
  ql/src/test/results/clientpositive/auto_join32.q.out f862870 
  ql/src/test/results/clientpositive/binarysortable_1.q.out 421fd2e 
  ql/src/test/results/clientpositive/cbo_subq_not_in.q.out c7274f7 
  ql/src/test/results/clientpositive/correlationoptimizer2.q.out c1a20c8 
  ql/src/test/results/clientpositive/correlationoptimizer6.q.out 6b003d5 
  ql/src/test/results/clientpositive/count.q.out e8d0cb3 
  ql/src/test/results/clientpositive/ctas_colname.q.out 232d505 
  ql/src/test/results/clientpositive/database.q.out 19b0ea2 
  ql/src/test/results/clientpositive/decimal_precision.q.out 8d67e2a 
  ql/src/test/results/clientpositive/decimal_udf.q.out 338b754 
  ql/src/test/results/clientpositive/distinct_stats.q.out 36049be 
  ql/src/test/results/clientpositive/dynpart_sort_opt_vectorization.q.out 
216a79c 
  ql/src/test/results/clientpositive/dynpart_sort_optimization.q.out 41049bd 
  
ql/src/test/results/clientpositive/encrypted/encryption_select_read_only_encrypted_tbl.q.out
 5d4d2af 
  ql/src/test/results/clientpositive/explain_logical.q.out 545034a 
  ql/src/test/results/clientpositive/fetch_aggregation.q.out 121d6a4 
  ql/src/test/results/clientpositive/gby_star.q.out a49f1bf 
  ql/src/test/results/clientpositive/groupby12.q.out 6eb3e96 
  ql/src/test/results/clientpositive/groupby5_map.q.out ea8ce4a 
  ql/src/test/results/clientpositive/groupby5_map_skew.q.out c0fced7 
  ql/src/test/results/clientpositive/groupby_cube1.q.out 62e9c54 
  ql/src/test/results/clientpositive/groupby_distinct_samekey.q.out bd92927 
  ql/src/test/results/clientpositive/groupby_grouping_sets2.q.out 6d44f32 
  ql/src/test/results/clientpositive/groupby_grouping_sets3.q.out 5d1a7a8 
  ql/src/test/results/clientpositive/groupby_grouping_sets5.q.out bd7c2ff 
  ql/src/test/results/clientpositive/groupby_grouping_sets6.q.out d807d44 
  ql/src/test/results/clientpositive/groupby_position.q.out 7152101 
  ql/src/test/results/clientpositive/groupby_resolution.q.out 7df53f2 
  ql/src/test/results/clientpositive/groupby_rollup1.q.out 8e04d86 
  ql/src/test/results/clientpositive/groupby_sort_10.q.out 2fead26 
  ql/src/test/results/clientpositive/groupby_sort_11.q.out adf585d 
  ql/src/test/results/clientpositive/groupby_sort_8.q.out ec16eb0 
  ql/src/test/results/clientpositive/having.q.out 8682a46 
  ql/src/test/results/clientpositive/having2.q.out aafd3b6 
  ql/src/test/results/clientpositive/index_auto_mult_tables.q.out 7755ed5 
  ql/src/test/results/clientpositive/index_auto_self_join.q.out a214845 
  ql/src/test/results/clientpositive/index_auto_update.q.out 70d0286 
  ql/src/test/results/clientpositive/index_bitmap_auto_partitioned.q.out 
65b6721 
  ql/src/test/results/clientpositive/index_bitmap_compression.q.out 9673f08 
  ql/src/test/results/clientpositive/infer_bucket_sort_dyn_part.q.out c2f0810 
  ql/src/test/results/clientpositive/infer_bucket_sort_map_operators.q.out 
6aef463 
  ql/src/test/results/clientpositive/join18.q.out c3ab306 
  ql/src/test/results/clientpositive/join18_multi_distinct.q.out a83102c 
  ql/src/test/results/clientpositive/join31.q.out 21aa1de 
  ql/src/test/results/clientpositive/limit_partition_metadataonly.q.out 904acfc 
  ql/src/test/results/clientpositive/limit_pushdown.q.out d77bc60 
  ql/src/test/results/clientpositive/lineage2.q.out 9b227c6 
  ql/src/test/results/clientpositive/lineage3.q.out b6b4e0b 
  ql/src/test/results/clientpositive/list_bucket_query_multiskew_3.q.out 
753729f 
  ql/src/test/results/clientpositive/mapjoin_mapjoin.q.out 9775f30 
  ql/src/test/results/clientpositive/metadata_only_queries.q.out 5907f4a 
  ql/src/test/results/clientpositive/metadataonly1.q.out 0d758a5 
  ql/src/test/results/clientpositive/multiMapJoin2.q.out 2cbef2e 
  ql/src/test/results/clientpositive/nonblock_op_deduplicate.q.out feadd5a 
  ql/src/test/results/clientpositive/nonmr_fetch.q.out a4ce905 
  ql/src/test/results/clientpositive/partition_multilevels.q.out 31862e1 
  ql/src/test/results/clientpositive/ppd_gby.q.out 87cb907 
  ql/src/test/results/clientpositive/ppd_gby2.q.out bc00149 
  ql/src/test/results/clientpositive/ppd_join_filter.q.out e99986f 
  ql/src/test/results/clientpositive/ql_rewrite_gbtoidx_cbo_1.q.out 7d1aff2 
  ql/src/test/results/clientpositive/ql_rewrite_gbtoidx_cbo_2.q.out 3ee2e0f 
  ql/src/test/results/clientpositive/reduce_deduplicate_extended.q.out df16ab2 
  ql/src/test/results/clientpositive/selectDistinctStar.q.out abd6795 
  ql/src/test/results/clientpositive/spark/auto_join18.q.out 8607221 
  ql/src/test/results/clientpositive/spark/auto_join18_multi_distinct.q.out 
df737c2 
  ql/src/test/results/clientpositive/spark/auto_join27.q.out 43313e0 
  ql/src/test/results/clientpositive/spark/auto_join32.q.out 316792b 
  ql/src/test/results/clientpositive/spark/cbo_subq_not_in.q.out c7274f7 
  ql/src/test/results/clientpositive/spark/count.q.out b2e9ffb 
  ql/src/test/results/clientpositive/spark/groupby5_map.q.out e775921 
  ql/src/test/results/clientpositive/spark/groupby5_map_skew.q.out 5c58934 
  ql/src/test/results/clientpositive/spark/groupby_cube1.q.out 25bfbec 
  ql/src/test/results/clientpositive/spark/groupby_position.q.out 088428b 
  ql/src/test/results/clientpositive/spark/groupby_resolution.q.out 9d36099 
  ql/src/test/results/clientpositive/spark/groupby_rollup1.q.out 0a06c65 
  ql/src/test/results/clientpositive/spark/having.q.out da59af9 
  
ql/src/test/results/clientpositive/spark/infer_bucket_sort_map_operators.q.out 
f39cd57 
  ql/src/test/results/clientpositive/spark/join18.q.out c702fdf 
  ql/src/test/results/clientpositive/spark/join18_multi_distinct.q.out 18d79d3 
  ql/src/test/results/clientpositive/spark/join31.q.out 108a1ea 
  ql/src/test/results/clientpositive/spark/limit_partition_metadataonly.q.out 
a34a399 
  ql/src/test/results/clientpositive/spark/limit_pushdown.q.out 2dc710c 
  ql/src/test/results/clientpositive/spark/mapjoin_mapjoin.q.out 3b2c80e 
  ql/src/test/results/clientpositive/spark/metadata_only_queries.q.out 18e1d4e 
  ql/src/test/results/clientpositive/spark/ppd_join_filter.q.out ed536c2 
  ql/src/test/results/clientpositive/spark/ql_rewrite_gbtoidx_cbo_1.q.out 
02e24c3 
  ql/src/test/results/clientpositive/spark/stats_only_null.q.out cb0920e 
  ql/src/test/results/clientpositive/spark/subquery_in.q.out 00b3399 
  ql/src/test/results/clientpositive/spark/union11.q.out a6b9367 
  ql/src/test/results/clientpositive/spark/union14.q.out 0c9542b 
  ql/src/test/results/clientpositive/spark/union15.q.out cb8bc75 
  ql/src/test/results/clientpositive/spark/union28.q.out 59657cc 
  ql/src/test/results/clientpositive/spark/union30.q.out fd907f9 
  ql/src/test/results/clientpositive/spark/union33.q.out 271c377 
  ql/src/test/results/clientpositive/spark/union5.q.out 06a5d90 
  ql/src/test/results/clientpositive/spark/union7.q.out 4a81283 
  ql/src/test/results/clientpositive/spark/union_remove_21.q.out 246594f 
  ql/src/test/results/clientpositive/spark/vector_count_distinct.q.out ee5462d 
  ql/src/test/results/clientpositive/spark/vector_decimal_aggregate.q.out 
7e161d1 
  ql/src/test/results/clientpositive/spark/vector_distinct_2.q.out 322270f 
  ql/src/test/results/clientpositive/spark/vector_groupby_3.q.out 3d6a236 
  ql/src/test/results/clientpositive/spark/vector_mapjoin_reduce.q.out a2dd910 
  ql/src/test/results/clientpositive/spark/vector_orderby_5.q.out 6bbb6b9 
  ql/src/test/results/clientpositive/spark/vectorization_0.q.out 3ad059c 
  ql/src/test/results/clientpositive/spark/vectorization_13.q.out 22be1d7 
  ql/src/test/results/clientpositive/spark/vectorization_15.q.out 8013bfe 
  ql/src/test/results/clientpositive/spark/vectorization_16.q.out a42c30a 
  ql/src/test/results/clientpositive/spark/vectorization_9.q.out a42c30a 
  ql/src/test/results/clientpositive/spark/vectorization_pushdown.q.out eb8914b 
  ql/src/test/results/clientpositive/spark/vectorization_short_regress.q.out 
3d17aba 
  ql/src/test/results/clientpositive/spark/vectorized_nested_mapjoin.q.out 
7ba64b7 
  ql/src/test/results/clientpositive/spark/vectorized_timestamp_funcs.q.out 
290db0d 
  ql/src/test/results/clientpositive/stats_only_null.q.out 55c5970 
  ql/src/test/results/clientpositive/stats_ppr_all.q.out 5f6f5d4 
  ql/src/test/results/clientpositive/subq_where_serialization.q.out c0b2a2d 
  ql/src/test/results/clientpositive/subquery_exists_having.q.out 14819c9 
  ql/src/test/results/clientpositive/subquery_in.q.out 7c53638 
  ql/src/test/results/clientpositive/subquery_in_having.q.out 8cabfa7 
  ql/src/test/results/clientpositive/subquery_notexists.q.out 81b4137 
  ql/src/test/results/clientpositive/subquery_notexists_having.q.out fd09901 
  ql/src/test/results/clientpositive/subquery_notin.q.out fd6d53b 
  ql/src/test/results/clientpositive/subquery_notin_having.q.java1.7.out 
775f477 
  ql/src/test/results/clientpositive/subquery_unqualcolumnrefs.q.out 5d6d4a8 
  ql/src/test/results/clientpositive/subquery_views.q.out 41834a3 
  ql/src/test/results/clientpositive/tez/cbo_subq_not_in.q.out c7274f7 
  ql/src/test/results/clientpositive/tez/count.q.out a5e6082 
  ql/src/test/results/clientpositive/tez/dynamic_partition_pruning.q.out 
cd2de52 
  ql/src/test/results/clientpositive/tez/dynpart_sort_opt_vectorization.q.out 
4451046 
  ql/src/test/results/clientpositive/tez/dynpart_sort_optimization.q.out 
cb001b9 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out 9756b0c 
  ql/src/test/results/clientpositive/tez/explainuser_2.q.out 71d8f41 
  ql/src/test/results/clientpositive/tez/having.q.out 80f02de 
  ql/src/test/results/clientpositive/tez/limit_pushdown.q.out 01ccae7 
  ql/src/test/results/clientpositive/tez/mapjoin_mapjoin.q.out 23df010 
  ql/src/test/results/clientpositive/tez/metadata_only_queries.q.out aaea52e 
  ql/src/test/results/clientpositive/tez/metadataonly1.q.out 927b686 
  ql/src/test/results/clientpositive/tez/mrr.q.out efbd02d 
  ql/src/test/results/clientpositive/tez/selectDistinctStar.q.out 0a091da 
  ql/src/test/results/clientpositive/tez/stats_only_null.q.out eabda89 
  ql/src/test/results/clientpositive/tez/subquery_in.q.out 38f4bc6 
  ql/src/test/results/clientpositive/tez/tez_dml.q.out ae85292 
  ql/src/test/results/clientpositive/tez/union5.q.out 327195c 
  ql/src/test/results/clientpositive/tez/union7.q.out c5ece62 
  ql/src/test/results/clientpositive/tez/unionDistinct_1.q.out cab3a28 
  ql/src/test/results/clientpositive/tez/vector_aggregate_9.q.out 382380e 
  ql/src/test/results/clientpositive/tez/vector_binary_join_groupby.q.out 
8dcd40d 
  ql/src/test/results/clientpositive/tez/vector_count_distinct.q.out 95863a3 
  ql/src/test/results/clientpositive/tez/vector_decimal_aggregate.q.out 2e9c232 
  ql/src/test/results/clientpositive/tez/vector_decimal_precision.q.out 4e1b654 
  ql/src/test/results/clientpositive/tez/vector_decimal_udf.q.out 13f0de2 
  ql/src/test/results/clientpositive/tez/vector_distinct_2.q.out 1671ddf 
  ql/src/test/results/clientpositive/tez/vector_groupby_3.q.out 9b9dcdd 
  ql/src/test/results/clientpositive/tez/vector_groupby_reduce.q.out ba2fb77 
  ql/src/test/results/clientpositive/tez/vector_grouping_sets.q.out 688d2ac 
  ql/src/test/results/clientpositive/tez/vector_mapjoin_reduce.q.out 29b86bd 
  ql/src/test/results/clientpositive/tez/vector_orderby_5.q.out af15925 
  ql/src/test/results/clientpositive/tez/vector_outer_join2.q.out f9159eb 
  ql/src/test/results/clientpositive/tez/vector_partition_diff_num_cols.q.out 
65b2ff1 
  ql/src/test/results/clientpositive/tez/vector_partitioned_date_time.q.out 
4b45ac1 
  ql/src/test/results/clientpositive/tez/vector_reduce_groupby_decimal.q.out 
528c2cb 
  ql/src/test/results/clientpositive/tez/vectorization_0.q.out 18e042d 
  ql/src/test/results/clientpositive/tez/vectorization_13.q.out 6214640 
  ql/src/test/results/clientpositive/tez/vectorization_15.q.out 1858cb0 
  ql/src/test/results/clientpositive/tez/vectorization_16.q.out bfed7d1 
  ql/src/test/results/clientpositive/tez/vectorization_9.q.out bfed7d1 
  ql/src/test/results/clientpositive/tez/vectorization_limit.q.out 1c5b51f 
  ql/src/test/results/clientpositive/tez/vectorization_pushdown.q.out e203bfd 
  ql/src/test/results/clientpositive/tez/vectorization_short_regress.q.out 
59b457a 
  ql/src/test/results/clientpositive/tez/vectorized_distinct_gby.q.out 23798e0 
  
ql/src/test/results/clientpositive/tez/vectorized_dynamic_partition_pruning.q.out
 b253508 
  ql/src/test/results/clientpositive/tez/vectorized_nested_mapjoin.q.out 
01a36e5 
  ql/src/test/results/clientpositive/tez/vectorized_parquet.q.out 8118139 
  ql/src/test/results/clientpositive/tez/vectorized_timestamp_funcs.q.out 
b285a4f 
  ql/src/test/results/clientpositive/udf8.q.out 8276ba9 
  ql/src/test/results/clientpositive/udf_count.q.out c9a596f 
  ql/src/test/results/clientpositive/union11.q.out 0d0b8fa 
  ql/src/test/results/clientpositive/union14.q.out 7a7e938 
  ql/src/test/results/clientpositive/union15.q.out 148f25f 
  ql/src/test/results/clientpositive/union28.q.out 38f8ee6 
  ql/src/test/results/clientpositive/union30.q.out 894ed49 
  ql/src/test/results/clientpositive/union33.q.out 308cd8b 
  ql/src/test/results/clientpositive/union5.q.out 75389f8 
  ql/src/test/results/clientpositive/union7.q.out 592f073 
  ql/src/test/results/clientpositive/unionDistinct_1.q.out 8d74fbe 
  ql/src/test/results/clientpositive/union_remove_21.q.out 4743d8d 
  ql/src/test/results/clientpositive/vector_aggregate_9.q.out f81816c 
  ql/src/test/results/clientpositive/vector_aggregate_without_gby.q.out 1175cb8 
  ql/src/test/results/clientpositive/vector_binary_join_groupby.q.out c3e4d52 
  ql/src/test/results/clientpositive/vector_count_distinct.q.out b87b2c7 
  ql/src/test/results/clientpositive/vector_decimal_aggregate.q.out 02c1c83 
  ql/src/test/results/clientpositive/vector_decimal_precision.q.out 16f2e3f 
  ql/src/test/results/clientpositive/vector_decimal_udf.q.out 91a585c 
  ql/src/test/results/clientpositive/vector_distinct_2.q.out 4153155 
  ql/src/test/results/clientpositive/vector_groupby_3.q.out af7cb58 
  ql/src/test/results/clientpositive/vector_groupby_reduce.q.out edcb0b3 
  ql/src/test/results/clientpositive/vector_grouping_sets.q.out 69d6187 
  ql/src/test/results/clientpositive/vector_left_outer_join.q.out 2f986c2 
  ql/src/test/results/clientpositive/vector_mapjoin_reduce.q.out 16916cb 
  ql/src/test/results/clientpositive/vector_orderby_5.q.out 45ccc62 
  ql/src/test/results/clientpositive/vector_outer_join1.q.out 8e8f7fa 
  ql/src/test/results/clientpositive/vector_outer_join2.q.out c649883 
  ql/src/test/results/clientpositive/vector_outer_join3.q.out 502f2d3 
  ql/src/test/results/clientpositive/vector_outer_join4.q.out 50ae497 
  ql/src/test/results/clientpositive/vector_outer_join5.q.out bbe8ba1 
  ql/src/test/results/clientpositive/vector_partition_diff_num_cols.q.out 
0437ff6 
  ql/src/test/results/clientpositive/vector_partitioned_date_time.q.out 83e7f19 
  ql/src/test/results/clientpositive/vector_reduce_groupby_decimal.q.out 
b38add3 
  ql/src/test/results/clientpositive/vectorization_0.q.out 89163cd 
  ql/src/test/results/clientpositive/vectorization_13.q.out 95cb09a 
  ql/src/test/results/clientpositive/vectorization_15.q.out da0e8e0 
  ql/src/test/results/clientpositive/vectorization_16.q.out 6ae3b4e 
  ql/src/test/results/clientpositive/vectorization_9.q.out 6ae3b4e 
  ql/src/test/results/clientpositive/vectorization_limit.q.out 7691a4d 
  ql/src/test/results/clientpositive/vectorization_pushdown.q.out 7205376 
  ql/src/test/results/clientpositive/vectorization_short_regress.q.out 728f628 
  ql/src/test/results/clientpositive/vectorized_distinct_gby.q.out 2e041a3 
  ql/src/test/results/clientpositive/vectorized_nested_mapjoin.q.out 82e90da 
  ql/src/test/results/clientpositive/vectorized_parquet.q.out 7b2dc6d 
  ql/src/test/results/clientpositive/vectorized_parquet_types.q.out 8d1bddc 
  ql/src/test/results/clientpositive/vectorized_timestamp_funcs.q.out f66903b 

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


Testing
-------


Thanks,

Ashutosh Chauhan

Reply via email to