-----------------------------------------------------------
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

Reply via email to