----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26209/#review55746 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/PlanModifierForASTConv.java <https://reviews.apache.org/r/26209/#comment96136> May be rename to isEmptyGrpAggr - John Pullokkaran On Oct. 8, 2014, 12:09 a.m., pengcheng xiong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26209/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2014, 12:09 a.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > create a derived table with new proj and aggr to address it > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/optiq/translator/PlanModifierForASTConv.java > 3d90ae7 > ql/src/test/queries/clientpositive/cbo_correctness.q f7f0722 > ql/src/test/results/clientpositive/cbo_correctness.q.out 3335d4d > ql/src/test/results/clientpositive/tez/cbo_correctness.q.out 5920612 > > Diff: https://reviews.apache.org/r/26209/diff/ > > > Testing > ------- > > > Thanks, > > pengcheng xiong > >
