----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72290/#review220164 -----------------------------------------------------------
LGTM, just a few comments. standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java Line 1300 (original), 1305 (patched) <https://reviews.apache.org/r/72290/#comment308483> Use setLong, setObject would require type inference standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java Lines 1390 (patched) <https://reviews.apache.org/r/72290/#comment308484> Not 100% sure, I think Laci P was working on removal of MIN_HISTORY_LEVEL table (HIVE-23107), please check with him. standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java Lines 4295 (patched) <https://reviews.apache.org/r/72290/#comment308485> Please extract this into org.apache.hadoop.hive.metastore.tools.SQLGenerator - Denys Kuzmenko On April 1, 2020, 6:53 a.m., Marton Bod wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72290/ > ----------------------------------------------------------- > > (Updated April 1, 2020, 6:53 a.m.) > > > Review request for hive, Denys Kuzmenko and Peter Vary. > > > Repository: hive-git > > > Description > ------- > > HIVE-23067: Use batch DB calls in TxnHandler for commitTxn and abortTxns > > > Diffs > ----- > > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java > 74ef88545e > > > Diff: https://reviews.apache.org/r/72290/diff/1/ > > > Testing > ------- > > Green build: https://builds.apache.org/job/PreCommit-HIVE-Build/21347/ > > > Thanks, > > Marton Bod > >
