> On May 19, 2020, 7:54 a.m., Peter Vary wrote: > >
Thank you for the review, fixed the issues + removed some parameters from the HMS.startMetaStore, since they were just implementation details unneccessary exposed. - Peter ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72480/#review220824 ----------------------------------------------------------- On May 20, 2020, 2:03 p.m., Peter Varga wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72480/ > ----------------------------------------------------------- > > (Updated May 20, 2020, 2:03 p.m.) > > > Review request for hive, Miklos Gergely and Peter Vary. > > > Repository: hive-git > > > Description > ------- > > Fix the timing to avoid flakyness. > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/MetastoreHousekeepingLeaderTestBase.java > a39a9c8e04 > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/MetastoreTaskThreadAlwaysTestImpl.java > 4cd2c58896 > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/RemoteMetastoreTaskThreadTestImpl1.java > c590b6aad5 > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/RemoteMetastoreTaskThreadTestImpl2.java > 5b50f66c51 > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetastoreHousekeepingLeader.java > 03a8161ea4 > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetastoreHousekeepingLeaderEmptyConfig.java > 75ea637503 > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetastoreHousekeepingNonLeader.java > 0341d3c03b > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 57c006b872 > > standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/MetaStoreTestUtils.java > 0e2c35acaa > > > Diff: https://reviews.apache.org/r/72480/diff/3/ > > > Testing > ------- > > > Thanks, > > Peter Varga > >
