Re: Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/ --- (Updated Jan. 25, 2018, 1:51 p.m.) Review request for hive, Marta Kuczora and

Re: Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/#review196234 --- Ship it! Ship It! - Peter Vary On Jan. 24, 2018, 2:02 p.m.,

Re: Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/#review196223 --- Ship it! Ship It! - Marta Kuczora On Jan. 24, 2018, 2:02

Re: Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/ --- (Updated Jan. 24, 2018, 2:02 p.m.) Review request for hive, Marta Kuczora and

Re: Review Request 65285: Create tests to cover getPartition(s) methods

> On Jan. 23, 2018, 4:12 p.m., Peter Vary wrote: > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetPartitions.java > > Lines 200 (patched) > > > > > > Maybe test for case

Re: Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/#review196018 --- Just 2 more test cases I can come up with. Is it worth to create

Re: Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/#review195995 --- Thanks a lot Adam for the patch. It looks very good to me. -

Review Request 65285: Create tests to cover getPartition(s) methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65285/ --- Review request for hive, Marta Kuczora and Peter Vary. Bugs: HIVE-18483