----------------------------------------------------------- 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