----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43115/#review117552 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/HiveGBOpConvUtil.java (line 225) <https://reviews.apache.org/r/43115/#comment178783> Instead of walking the expression you could keep a map of inputref to exprnode. - John Pullokkaran On Feb. 2, 2016, 10:16 p.m., Hari Sankar Sivarama Subramaniyan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43115/ > ----------------------------------------------------------- > > (Updated Feb. 2, 2016, 10:16 p.m.) > > > Review request for hive, Jesús Camacho Rodríguez and John Pullokkaran. > > > Repository: hive-git > > > Description > ------- > > CBO: Calcite Operator To Hive Operator (Calcite Return Path): TestCliDriver > groupby_ppr_multi_distinct.q failure > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/HiveGBOpConvUtil.java > 7fbf8cd > > Diff: https://reviews.apache.org/r/43115/diff/ > > > Testing > ------- > > Precommit runs > > > Thanks, > > Hari Sankar Sivarama Subramaniyan > >
