----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67399/#review204139 -----------------------------------------------------------
Thanks for the tests. Can we add a one more test which repeats the same test accross multiple sessions (can be simulated by using objectstore instances)? So for example: 1. Session 1 : list partitions with jdo, Session 2 : list partitions with jdo 2. Session 1 : drop partitions with directSQL 3. Session 2 : list partitions with jdo standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java Lines 2562 (patched) <https://reviews.apache.org/r/67399/#comment286535> why do we need it to be protected? Can we use package private instead? - Vihang Karajgaonkar On May 31, 2018, 1:53 p.m., Peter Vary wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67399/ > ----------------------------------------------------------- > > (Updated May 31, 2018, 1:53 p.m.) > > > Review request for hive, Alexander Kolbasov and Vihang Karajgaonkar. > > > Bugs: HIVE-19503 > https://issues.apache.org/jira/browse/HIVE-19503 > > > Repository: hive-git > > > Description > ------- > > The patch contains 2 tests: > > Test which checks if the JDO cache is able to handle directSql partition drops > Test which checks if the directSQL partition drop removes every connected > data from the RDBMS tables. > To create these tests we have 2 helper methods: > > Method to create the partitioned table > Method to check the number of rows in a given RDBMS table > Added a new ObjectStore.dropPartitionsInternal method which only visible for > testing so we can make sure that the dropPartition is using directSql and > does not fall back to JDO. > > Fixed a problem where some of the tables are not created automatically by the > tests, adding new init queries to MetaStoreDirectSql.ensureDbInit method > > > Diffs > ----- > > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java > 5bb1985 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java > b15d89d > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/TestObjectStore.java > 9912213 > > > Diff: https://reviews.apache.org/r/67399/diff/1/ > > > Testing > ------- > > Run the new tests > > > Thanks, > > Peter Vary > >
