> On April 23, 2018, 7:24 p.m., Gopal V wrote: > > ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java > > Lines 356 (patched) > > <https://reviews.apache.org/r/66485/diff/8/?file=2010487#file2010487line358> > > > > Is that supposed to be a "," or a +?
, - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66485/#review201756 ----------------------------------------------------------- On April 23, 2018, 7:03 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66485/ > ----------------------------------------------------------- > > (Updated April 23, 2018, 7:03 p.m.) > > > Review request for hive and Eugene Koifman. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 2403d7ac6c > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/txn/compactor/TestCompactor.java > 82ba775286 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java a35a215bfc > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 4e10649136 > ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java dde20ed56e > ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java > b1c2288d01 > ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Initiator.java > 22765b8e63 > ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Worker.java fe0aaa4ff5 > ql/src/test/org/apache/hadoop/hive/ql/TestTxnCommandsForMmTable.java > c053860b36 > > standalone-metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/hive_metastoreConstants.java > cb1d40a4a8 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnUtils.java > 7b02865e18 > > storage-api/src/java/org/apache/hadoop/hive/common/ValidReaderWriteIdList.java > 107ea9028a > > > Diff: https://reviews.apache.org/r/66485/diff/8/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
