----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25395/#review52521 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java <https://reviews.apache.org/r/25395/#comment91319> there's a new case now that has appmastereventoperator as a child of group by. ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java <https://reviews.apache.org/r/25395/#comment91320> we're doing map side aggr which yields different cardinalities for each set. Shouldn't the code account for that? - Gunther Hagleitner On Sept. 5, 2014, 6:58 p.m., Prasanth_J wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25395/ > ----------------------------------------------------------- > > (Updated Sept. 5, 2014, 6:58 p.m.) > > > Review request for hive, Ashutosh Chauhan, Gunther Hagleitner, and Harish > Butani. > > > Bugs: HIVE-7990 > https://issues.apache.org/jira/browse/HIVE-7990 > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java > 01c1d30 > ql/src/test/queries/clientpositive/annotate_stats_groupby.q e8e84c6 > ql/src/test/results/clientpositive/annotate_stats_groupby.q.out 9c37d9b > ql/src/test/results/clientpositive/groupby_cube1.q.out 4246744 > ql/src/test/results/clientpositive/groupby_grouping_sets2.q.out 1cd65f6 > ql/src/test/results/clientpositive/groupby_grouping_sets3.q.out f1ecbb4 > ql/src/test/results/clientpositive/groupby_rollup1.q.out 5db5cd5 > > Diff: https://reviews.apache.org/r/25395/diff/ > > > Testing > ------- > > > Thanks, > > Prasanth_J > >
