----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66485/#review201290 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java Lines 352 (patched) <https://reviews.apache.org/r/66485/#comment282440> Table type? ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java Lines 1119 (patched) <https://reviews.apache.org/r/66485/#comment282441> add comment about this (As the "file that adds this commit"). storage-api/src/java/org/apache/hadoop/hive/common/ValidReaderWriteIdList.java Lines 255 (patched) <https://reviews.apache.org/r/66485/#comment282439> Should probably return a new Object here (for sane debugging). - Gopal V On April 16, 2018, 10:35 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66485/ > ----------------------------------------------------------- > > (Updated April 16, 2018, 10:35 p.m.) > > > Review request for hive and Eugene Koifman. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > > 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 a88453c978 > 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 > > 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/3/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >