----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69633/#review211722 -----------------------------------------------------------
Thanks for the fix! Fix it and ship it! ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverSkewJoin.java Line 121 (original), 125 (patched) <https://reviews.apache.org/r/69633/#comment297277> Could you please remove the commented out line? - Peter Vary On jan. 5, 2019, 5:50 de, MANI M wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69633/ > ----------------------------------------------------------- > > (Updated jan. 5, 2019, 5:50 de) > > > Review request for hive, Peter Vary and Vihang Karajgaonkar. > > > Bugs: HIVE-20159 > https://issues.apache.org/jira/browse/HIVE-20159 > > > Repository: hive-git > > > Description > ------- > > Do Not Print StackTraces to STDERR in ConditionalResolverSkewJoin HIVE-20159 > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverSkewJoin.java > 5d25007d77 > > > Diff: https://reviews.apache.org/r/69633/diff/1/ > > > Testing > ------- > > Patch Uploaded to JIRA HIVE-20159, PRE-COMMIT testing completed. > https://issues.apache.org/jira/browse/HIVE-20159 > > > Thanks, > > MANI M > >