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

(Updated May 4, 2017, 7:43 p.m.)


Review request for hive and Jason Dere.


Changes
-------

Implemented the review comment.


Repository: hive-git


Description
-------

Semijoin Hints should use column name, if provided for partition key check.
Involves some code refactoring.


Diffs (updated)
-----

  
ql/src/java/org/apache/hadoop/hive/ql/optimizer/DynamicPartitionPruningOptimization.java
 b8c01020b7 
  ql/src/test/queries/clientpositive/semijoin_hint.q a3cd1d664d 
  ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction.q.out 
1d1f86bfaa 
  ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction_2.q.out 
a5fdd90811 
  ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction_3.q.out 
8950b70b09 
  ql/src/test/results/clientpositive/llap/dynamic_semijoin_user_level.q.out 
b910df4f34 
  ql/src/test/results/clientpositive/llap/semijoin_hint.q.out 388888ef1c 


Diff: https://reviews.apache.org/r/58973/diff/4/

Changes: https://reviews.apache.org/r/58973/diff/3-4/


Testing
-------


Thanks,

Deepak Jaiswal

Reply via email to