----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71243/#review217752 -----------------------------------------------------------
Fix it and ship it ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java Lines 1524-1529 (patched) <https://reviews.apache.org/r/71243/#comment305117> Please move these check after the assertTempTablePartitioned ql/src/test/org/apache/hadoop/hive/ql/metadata/TestSessionHiveMetastoreClientDropPartitionsTempTable.java Lines 139-144 (patched) <https://reviews.apache.org/r/71243/#comment305118> Please ensure that for Temp table we are working in the same way than for normal tables service/src/java/org/apache/hive/service/cli/thrift/ThreadPoolExecutorWithOomHook.java Line 57 (original) <https://reviews.apache.org/r/71243/#comment305119> Please check this change - Peter Vary On aug. 7, 2019, 8:35 de, Laszlo Pinter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71243/ > ----------------------------------------------------------- > > (Updated aug. 7, 2019, 8:35 de) > > > Review request for hive, Marta Kuczora, Peter Vary, and Adam Szita. > > > Repository: hive-git > > > Description > ------- > > HIVE-21875: Implement drop partition related methods on temporary tables. > > This is one of the subtasks for HIVE-21765, to support partitions on > temporary tables. > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java > a2c84b4620fb1eb90069e294204f604565ffed9b > > ql/src/test/org/apache/hadoop/hive/ql/metadata/TestSessionHiveMetastoreClientDropPartitionsTempTable.java > PRE-CREATION > > service/src/java/org/apache/hive/service/cli/thrift/ThreadPoolExecutorWithOomHook.java > 129413681045d790ae20bf4d8060f04162224565 > > standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestDropPartitions.java > 91c9edac95e9f9688dd2b806aeeb0823af02574f > > > Diff: https://reviews.apache.org/r/71243/diff/1/ > > > Testing > ------- > > Unit testing is done via > TestSessionHiveMetastoreClientDropPartitionsTempTable.java > > > Thanks, > > Laszlo Pinter > >
