----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66800/ -----------------------------------------------------------
(Updated May 11, 2018, 2:13 p.m.) Review request for hive, Alexander Kolbasov, Alan Gates, Marta Kuczora, Adam Szita, and Vihang Karajgaonkar. Changes ------- Addressed Sasha's comments Bugs: HIVE-6980 https://issues.apache.org/jira/browse/HIVE-6980 Repository: hive-git Description ------- First version of the patch. Splits getPartitionsViaSqlFilterInternal to: getPartitionIdsViaSqlFilter - which returns the partition ids getPartitionsFromPartitionIds - which returns the partition data for the partitions Creates dropPartitionsByPartitionIds which drops the partitions by directSQL commands Creates a dropPartitionsViaSqlFilter using getPartitionIdsViaSqlFilter and dropPartitionsByPartitionIds. Modifies the ObjectStore to drop partitions with directsql if possible. Diffs (updated) ----- standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java 56fbfed standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java b0a805f Diff: https://reviews.apache.org/r/66800/diff/4/ Changes: https://reviews.apache.org/r/66800/diff/3-4/ Testing ------- Run the TestDropPartition tests, also checked the database manually, that no object left in the database Thanks, Peter Vary