----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58585/#review172548 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java Line 829 (original) <https://reviews.apache.org/r/58585/#comment245650> Just a nit, but why was removeSemiJoin removed? Seems like we are now duplicating the logic to remove the semijoin. - Jason Dere On April 20, 2017, 6:20 p.m., Deepak Jaiswal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58585/ > ----------------------------------------------------------- > > (Updated April 20, 2017, 6:20 p.m.) > > > Review request for hive, Jason Dere and Prasanth_J. > > > Bugs: HIVE-16441 > https://issues.apache.org/jira/browse/HIVE-16441 > > > Repository: hive-git > > > Description > ------- > > De-duplicate semijoin branches in n-way joins > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/DynamicPartitionPruningOptimization.java > eb3eba536c > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java 9a69f90483 > ql/src/java/org/apache/hadoop/hive/ql/parse/TaskCompiler.java 96525b4b66 > ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java 26eda0461f > ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction.q.out > a47ce6e583 > > > Diff: https://reviews.apache.org/r/58585/diff/1/ > > > Testing > ------- > > > Thanks, > > Deepak Jaiswal > >
