You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com> on 2016/01/07 00:39:58 UTC
Review Request 42000: HIVE-12645 : ConstantPropagateProcCtx.resolve()
should
verify internal names in addition to alias to match 2 columns from
different row schemas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42000/
-----------------------------------------------------------
Review request for hive, Ashutosh Chauhan and John Pullokkaran.
Repository: hive-git
Description
-------
ConstantPropagateProcCtx.resolve() should verify internal names in addition to alias to match 2 columns from different row schemas
Diffs
-----
ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagateProcCtx.java 0a61f12
ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java 8da9ad5
ql/src/test/results/clientpositive/bucket_groupby.q.out 0e30801
ql/src/test/results/clientpositive/dynamic_rdd_cache.q.out 66d9a78
ql/src/test/results/clientpositive/dynpart_sort_optimization2.q.out 141bcd8
ql/src/test/results/clientpositive/groupby_ppd.q.out e3e4a50
ql/src/test/results/clientpositive/groupby_sort_1_23.q.out ceecbb9
ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 009ab2e
ql/src/test/results/clientpositive/list_bucket_query_oneskew_2.q.out 48cc6ea
ql/src/test/results/clientpositive/partition_multilevels.q.out 0fe7f82
ql/src/test/results/clientpositive/perf/query31.q.out 8c370c4
ql/src/test/results/clientpositive/perf/query39.q.out f4ad98c
ql/src/test/results/clientpositive/perf/query42.q.out 94f7e32
ql/src/test/results/clientpositive/perf/query52.q.out b4f46cc
ql/src/test/results/clientpositive/perf/query64.q.out 68730e7
ql/src/test/results/clientpositive/perf/query66.q.out b2e6bf7
ql/src/test/results/clientpositive/quotedid_basic.q.out 29736af
ql/src/test/results/clientpositive/quotedid_partition.q.out e40d0d0
ql/src/test/results/clientpositive/spark/dynamic_rdd_cache.q.out 1f26724
ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out fb75a98
ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out b9fb6b1
ql/src/test/results/clientpositive/tez/dynpart_sort_optimization2.q.out 8ef2b06
ql/src/test/results/clientpositive/tez/vector_null_projection.q.out 88587e9
ql/src/test/results/clientpositive/vector_null_projection.q.out c787fd4
Diff: https://reviews.apache.org/r/42000/diff/
Testing
-------
Thanks,
Hari Sankar Sivarama Subramaniyan