> On March 7, 2018, 5:06 p.m., Sahil Takiar wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > > Lines 1006 (patched) > > <https://reviews.apache.org/r/65952/diff/1/?file=1972255#file1972255line1006> > > > > Do the other parameters need to be checked? > > Marta Kuczora wrote: > You are right, we can check the other parameters too.
Added checks to HiveMetaStore for the db and table names and for the part_vals list. - Marta ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65952/#review198796 ----------------------------------------------------------- On March 8, 2018, 9:35 a.m., Marta Kuczora wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65952/ > ----------------------------------------------------------- > > (Updated March 8, 2018, 9:35 a.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/HiveMetaStore.java > 662de9a > > 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/2/ > > > Testing > ------- > > Run the TestDropPartitions tests. > > > Thanks, > > Marta Kuczora > >
