----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71550/#review218017 -----------------------------------------------------------
Ship it! Just one minor comment, we can fix that in a follow-up jira ql/src/test/org/apache/hadoop/hive/ql/metadata/TestSessionHiveMetastoreClientAlterPartitionsTempTable.java Lines 183-224 (patched) <https://reviews.apache.org/r/71550/#comment305483> I think in a follow-up jira we should move these test to the original TestAlterPartition test. - Peter Vary On szept. 26, 2019, 4:54 du, Laszlo Pinter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71550/ > ----------------------------------------------------------- > > (Updated szept. 26, 2019, 4:54 du) > > > Review request for hive, Marta Kuczora, Peter Vary, and Adam Szita. > > > Repository: hive-git > > > Description > ------- > > HIVE-22137: Implement alter/rename partition related methods on temporary > tables > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/metadata/PartitionTree.java > c84c3ef595a5f26232d4f003e46f74bb14a7ec99 > > ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java > a5b16d12a514439046929dde031bbe8e80f71a28 > ql/src/java/org/apache/hadoop/hive/ql/metadata/TempTable.java > fa6dddcbadec59579357b9bc3d4ea42e44a1ca6f > > ql/src/test/org/apache/hadoop/hive/ql/metadata/TestSessionHiveMetastoreClientAlterPartitionsTempTable.java > PRE-CREATION > > standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java > 6c7d80ea576d6b10af191d8ceb158c20b1f70b46 > > standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestAlterPartitions.java > 4fc3688f2e5c06e254e2177dbf142b424dcfddd8 > > > Diff: https://reviews.apache.org/r/71550/diff/2/ > > > Testing > ------- > > > Thanks, > > Laszlo Pinter > >
