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