[hive] HiveCatalog should not delete dir for drop partition

Author: JingsongLi

paimon-hive/paimon-hive-catalog/src/main/java/org/apache/paimon/hive/HiveCatalog.java

@@ -395,9 +395,6 @@ public void dropPartitions(Identifier identifier, List<Map<String, String>> part
                     tagToPart
                             ? partitions
                             : removePartitionsExistsInOtherBranches(identifier, partitions);
-            Table hmsTable = getHmsTable(identifier);
-            boolean externalTable = isExternalTable(hmsTable);
-            String dataFilePath = getDataFilePath(identifier, hmsTable);
             for (Map<String, String> part : metaPartitions) {
                 List<String> partitionValues = new ArrayList<>(part.values());
                 try {
@@ -408,21 +405,6 @@ public void dropPartitions(Identifier identifier, List<Map<String, String>> part
                                             identifier.getTableName(),
                                             partitionValues,
                                             false));
-
-                    if (!externalTable) {
-                        Path partitionLocation = new Path(getPartitionLocation(dataFilePath, part));
-                        try {
-                            if (fileIO.exists(partitionLocation)) {
-                                fileIO.deleteDirectoryQuietly(partitionLocation);
-                            }
-                        } catch (Exception ee) {
-                            LOG.error(
-                                    "Delete directory[{}] fail for table {} partition.",
-                                    partitionLocation,
-                                    identifier,
-                                    ee);
-                        }
-                    }
                 } catch (NoSuchObjectException e) {
                     // do nothing if the partition not exists
                 } catch (Exception e) {

paimon-hive/paimon-hive-connector-common/src/test/java/org/apache/paimon/hive/HiveCatalogITCaseBase.java

@@ -606,12 +606,12 @@ private void testDropPartitionFromBranchImpl() throws Exception {
         tEnv.executeSql("ALTER TABLE `t$branch_test` DROP PARTITION (pt = 3)");
         assertThat(hiveShell.executeQuery("SHOW PARTITIONS t"))
                 .containsExactlyInAnyOrder("pt=1", "pt=2", "pt=4");
-        assertThat(fileIO.exists(new Path(tablePath, "pt=3"))).isFalse();
+        assertThat(fileIO.exists(new Path(tablePath, "pt=3"))).isTrue();
 
         tEnv.executeSql("ALTER TABLE t DROP PARTITION (pt = 1)");
         assertThat(hiveShell.executeQuery("SHOW PARTITIONS t"))
                 .containsExactlyInAnyOrder("pt=2", "pt=4");
-        assertThat(fileIO.exists(new Path(tablePath, "pt=1"))).isFalse();
+        assertThat(fileIO.exists(new Path(tablePath, "pt=1"))).isTrue();
 
         tEnv.executeSql("ALTER TABLE t DROP PARTITION (pt = 4)");
         assertThat(hiveShell.executeQuery("SHOW PARTITIONS t"))
@@ -620,11 +620,11 @@ private void testDropPartitionFromBranchImpl() throws Exception {
 
         tEnv.executeSql("ALTER TABLE `t$branch_test` DROP PARTITION (pt = 4)");
         assertThat(hiveShell.executeQuery("SHOW PARTITIONS t")).containsExactlyInAnyOrder("pt=2");
-        assertThat(fileIO.exists(new Path(tablePath, "pt=4"))).isFalse();
+        assertThat(fileIO.exists(new Path(tablePath, "pt=4"))).isTrue();
 
         tEnv.executeSql("ALTER TABLE t DROP PARTITION (pt = 2)");
         assertThat(hiveShell.executeQuery("SHOW PARTITIONS t")).isEmpty();
-        assertThat(fileIO.exists(new Path(tablePath, "pt=2"))).isFalse();
+        assertThat(fileIO.exists(new Path(tablePath, "pt=2"))).isTrue();
     }
 
     @Test