----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41582/#review111501 -----------------------------------------------------------
Ship it! Ship It! - Xuefu Zhang On Dec. 21, 2015, 3:46 p.m., Chaoyu Tang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41582/ > ----------------------------------------------------------- > > (Updated Dec. 21, 2015, 3:46 p.m.) > > > Review request for hive. > > > Bugs: HIVE-12713 > https://issues.apache.org/jira/browse/HIVE-12713 > > > Repository: hive-git > > > Description > ------- > > Patch improves the driver compile and execute logging in following: > 1. ensuring that only the redacted query to be logged out > 2. removing redundant variable substitution in HS2 SQLOperation > 3. logging out the query and its compilation time without having to enable > PerfLogger debug, to help identify badly written queries which take a lot of > time to compile and probably cause other good queries to be queued > (HIVE-12516) > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java 98ebd50 > > itests/hive-unit/src/test/java/org/apache/hive/beeline/TestBeeLineWithArgs.java > 7cc0acf > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcDriver2.java > e9206b9 > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/cbo_rp_TestJdbcDriver2.java > c66f166 > > itests/hive-unit/src/test/java/org/apache/hive/service/cli/operation/TestOperationLoggingAPIWithMr.java > d21571e > > itests/hive-unit/src/test/java/org/apache/hive/service/cli/operation/TestOperationLoggingAPIWithTez.java > 8b21647 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java 3d5f3b5 > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseDriver.java c33bb66 > service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java > d90dd0d > > Diff: https://reviews.apache.org/r/41582/diff/ > > > Testing > ------- > > 1. Manual tests > 2. Precommit tests > > > Thanks, > > Chaoyu Tang > >