----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16026/ -----------------------------------------------------------
(Updated Dec. 5, 2013, 5:06 a.m.) Review request for hive and Navis Ryu. Changes ------- Addressed comments. Bugs: HIVE-5916 https://issues.apache.org/jira/browse/HIVE-5916 Repository: hive Description ------- No need to aggregate statistics collected via counter mechanism Diffs (updated) ----- trunk/itests/qtest/pom.xml 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRTableScan1.java 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 1548014 trunk/ql/src/java/org/apache/hadoop/hive/ql/stats/CounterStatsAggregator.java 1548014 trunk/ql/src/test/queries/clientpositive/stats_counter_partitioned.q PRE-CREATION trunk/ql/src/test/results/clientpositive/stats12.q.out 1548014 trunk/ql/src/test/results/clientpositive/stats13.q.out 1548014 trunk/ql/src/test/results/clientpositive/stats_counter_partitioned.q.out PRE-CREATION Diff: https://reviews.apache.org/r/16026/diff/ Testing ------- Added a new test case. Ran all other stats test. Thanks, Ashutosh Chauhan