----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65952/#review198796 -----------------------------------------------------------
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java Lines 996 (patched) <https://reviews.apache.org/r/65952/#comment279042> Why do this validation in `HiveMetaStoreClient` rather than `HiveMetaStore`? standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java Lines 1006 (patched) <https://reviews.apache.org/r/65952/#comment279043> Do the other parameters need to be checked? - Sahil Takiar On March 7, 2018, 3:48 p.m., Marta Kuczora wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65952/ > ----------------------------------------------------------- > > (Updated March 7, 2018, 3:48 p.m.) > > > Review request for hive, Peter Vary and Adam Szita. > > > Bugs: HIVE-18898 > https://issues.apache.org/jira/browse/HIVE-18898 > > > Repository: hive-git > > > Description > ------- > > The TestDropPartitions tests revealed that NPE is thrown if the > dropPartition(String db_name, String tbl_name, List<String> part_vals, > PartitionDropOptions options) method is called with null options and with a > part_vals list which contains null elements. > > > Diffs > ----- > > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > 3128089 > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestDropPartitions.java > 4d94ebf > > > Diff: https://reviews.apache.org/r/65952/diff/1/ > > > Testing > ------- > > Run the TestDropPartitions tests. > > > Thanks, > > Marta Kuczora > >
