-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15993/
-----------------------------------------------------------
(Updated Dec. 9, 2013, 5:28 a.m.)
Review request for hive and Ashutosh Chauhan.
Changes
-------
Minor refactorings to simplify things
Bugs: HIVE-5936
https://issues.apache.org/jira/browse/HIVE-5936
Repository: hive-git
Description
-------
With counter mechanism, MR job is successful, but StatsTask on client fails
with NPE
Diffs (updated)
-----
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
d0e94bf
metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java c43145b
ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java cbc3cd2
ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java bd2c4a8
ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java a2ecc80
ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java 46d88ce
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 197a20f
ql/src/java/org/apache/hadoop/hive/ql/io/rcfile/stats/PartialScanMapper.java
7e701f4
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRFileSink1.java cca8481
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRTableScan1.java af729e6
ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 08b9e58
ql/src/java/org/apache/hadoop/hive/ql/stats/CounterStatsAggregator.java
fa430eb
ql/src/java/org/apache/hadoop/hive/ql/stats/StatsFactory.java 8ae32f0
ql/src/java/org/apache/hadoop/hive/ql/stats/jdbc/JDBCStatsPublisher.java
8c23b87
ql/src/test/queries/clientpositive/stats_counter.q 20769e4
ql/src/test/queries/clientpositive/stats_noscan_2.q a19d01b
ql/src/test/results/clientpositive/stats_counter.q.out f15d8c5
ql/src/test/results/clientpositive/stats_noscan_1.q.out 054ca38
ql/src/test/results/clientpositive/stats_noscan_2.q.out e55fa94
Diff: https://reviews.apache.org/r/15993/diff/
Testing
-------
Thanks,
Navis Ryu