Indhumathi27 commented on code in PR #4915:
URL: https://github.com/apache/hive/pull/4915#discussion_r1418668565


##########
ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java:
##########
@@ -5542,6 +5542,13 @@ private void replaceFiles(Path tablePath, Path srcf, 
Path destf, Path oldPath, H
         throw new HiveException("Getting globStatus " + srcf.toString(), e);
       }
 
+      // in insert/load overwrite table query for partitions not in metastore 
but exist in filesystem,

Review Comment:
   ```suggestion
               // For insert/load overwrite cases, where external.table.purge 
is disabled for the table, there may be stable
         // partitions present in the table location after Alter DROP Partition 
operation. In such cases, oldPath will be
         // null, since those partitions will not be present in metastore. 
Added below check to clean up those stale partitions.
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to