----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27569/#review59768 -----------------------------------------------------------
Ship it! Ship It! - Xuefu Zhang On Nov. 4, 2014, 12:35 p.m., chengxiang li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27569/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2014, 12:35 p.m.) > > > Review request for hive, Rui Li and Xuefu Zhang. > > > Bugs: HIVE-8670 > https://issues.apache.org/jira/browse/HIVE-8670 > > > Repository: hive-git > > > Description > ------- > > add job statistic API and constructor logic, still miss spark task metric > collection logic. > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkTask.java 04323bb > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/Statistic/SparkStatistic.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/Statistic/SparkStatisticGroup.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/Statistic/SparkStatistics.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/Statistic/SparkStatisticsBuilder.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/status/SparkJobStatus.java > a450af4 > > ql/src/java/org/apache/hadoop/hive/ql/exec/spark/status/impl/SimpleSparkJobStatus.java > db2eca4 > > Diff: https://reviews.apache.org/r/27569/diff/ > > > Testing > ------- > > > Thanks, > > chengxiang li > >