> On May 2, 2017, 8:15 p.m., Jason Dere wrote: > > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java > > Lines 11192 (patched) > > <https://reviews.apache.org/r/58914/diff/1/?file=1705389#file1705389line11200> > > > > Can you give the method name something different than OldHints - how > > about getHintsFromQB?
sure. > On May 2, 2017, 8:15 p.m., Jason Dere wrote: > > ql/src/test/queries/clientpositive/semijoin_hint.q > > Lines 46 (patched) > > <https://reviews.apache.org/r/58914/diff/1/?file=1705394#file1705394line46> > > > > Does this query, with no hints, have any runtime filtering? Otherwise > > you wouldn't be able to distinguish whether the None option works or not. Yes it does. > On May 2, 2017, 8:15 p.m., Jason Dere wrote: > > ql/src/test/results/clientpositive/llap/semijoin_hint.q.out > > Lines 233 (patched) > > <https://reviews.apache.org/r/58914/diff/1/?file=1705395#file1705395line233> > > > > Should the expectedEntries be 5000 per the hint? Thanks for brining this up. It is happening incase of return path true. Investigating. > On May 2, 2017, 8:15 p.m., Jason Dere wrote: > > ql/src/test/results/clientpositive/llap/semijoin_hint.q.out > > Lines 266 (patched) > > <https://reviews.apache.org/r/58914/diff/1/?file=1705395#file1705395line266> > > > > Should the expectedEntries be 5000 per the hint? Yes, looking into it. > On May 2, 2017, 8:15 p.m., Jason Dere wrote: > > ql/src/test/results/clientpositive/llap/semijoin_hint.q.out > > Lines 300 (patched) > > <https://reviews.apache.org/r/58914/diff/1/?file=1705395#file1705395line300> > > > > Should the expectedEntries be 5000 per the hint? ditto - Deepak ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58914/#review173628 ----------------------------------------------------------- On May 2, 2017, 5:47 a.m., Deepak Jaiswal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58914/ > ----------------------------------------------------------- > > (Updated May 2, 2017, 5:47 a.m.) > > > Review request for hive, Jason Dere and Sergey Shelukhin. > > > Bugs: HIVE-16550 > https://issues.apache.org/jira/browse/HIVE-16550 > > > Repository: hive-git > > > Description > ------- > > Semijoin Hints should be able to skip the optimization if needed. > In addition to that, the patch fixes several issues with hints in general > such as, > - It now works with subqueries and unions. > - Uses a global data structure instead of per QB. > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/DynamicPartitionPruningOptimization.java > e1a69526bc > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/HiveOpConverter.java > d375d1b58d > ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 1b054a7e24 > ql/src/java/org/apache/hadoop/hive/ql/parse/HintParser.g e110fb33df > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java 3a1f821bd3 > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseUtils.java 54e37f7c80 > ql/src/java/org/apache/hadoop/hive/ql/parse/QBParseInfo.java 7bf1c599a5 > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java > 654f3b1772 > ql/src/java/org/apache/hadoop/hive/ql/parse/TaskCompiler.java 5ea7800528 > ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeDynamicListDesc.java > 3143554ec6 > ql/src/java/org/apache/hadoop/hive/ql/plan/JoinDesc.java 032c7bb28d > ql/src/java/org/apache/hadoop/hive/ql/ppd/SyntheticJoinPredicate.java > f45daa8828 > ql/src/test/queries/clientpositive/semijoin_hint.q 5de0c8c8c1 > ql/src/test/results/clientpositive/llap/semijoin_hint.q.out bc248930ec > > > Diff: https://reviews.apache.org/r/58914/diff/1/ > > > Testing > ------- > > > Thanks, > > Deepak Jaiswal > >
