----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/948/#review909 -----------------------------------------------------------
trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java <https://reviews.apache.org/r/948/#comment1962> Because I use this in finally block. trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java <https://reviews.apache.org/r/948/#comment1963> What's the concern? trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java <https://reviews.apache.org/r/948/#comment1964> We might not. If it doesn't support it will return null. That's why I have a null logic next to ignore null results and don't record anything. trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java <https://reviews.apache.org/r/948/#comment1965> Yes. trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java <https://reviews.apache.org/r/948/#comment1966> Why? - Siying On 2011-06-23 07:41:33, Siying Dong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/948/ > ----------------------------------------------------------- > > (Updated 2011-06-23 07:41:33) > > > Review request for hive, Yongqiang He, Ning Zhang, and namit jain. > > > Summary > ------- > > In hive CLI, print out CPU msec from Hadoop MapReduce coutners. > > > This addresses bug HIVE-2236. > https://issues.apache.org/jira/browse/HIVE-2236 > > > Diffs > ----- > > trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java 1138748 > trunk/ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java 1138748 > trunk/ql/src/java/org/apache/hadoop/hive/ql/MapRedStats.java PRE-CREATION > trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ExecDriver.java 1138748 > trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java > 1138748 > trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 1138748 > trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java > 1138748 > > Diff: https://reviews.apache.org/r/948/diff > > > Testing > ------- > > run the updated codes against real clusters and make sure it printing is > correct. > > > Thanks, > > Siying > >
