vineetgarg02 commented on a change in pull request #552: Hive 21279
URL: https://github.com/apache/hive/pull/552#discussion_r260981138
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java
##########
@@ -1511,9 +1525,10 @@ public static void mvFileToFinalPath(Path specPath,
Configuration hconf,
// Deduplicate and keep a list of files
// Pass on the list of files to conf (to be used later by fetch
operator)
// if true:
- // Rename tmpPath to a new directory name
- // Remove duplicates
- // Rename/move the new directory to specPath
+ // 1) Rename tmpPath to a new directory name to prevent additional
files
Review comment:
@ashutoshc the earlier comment does not make sense anymore. for blobstore FS
there is no step number 2. What comment line are you referring to?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
