rbalamohan commented on a change in pull request #3058:
URL: https://github.com/apache/hive/pull/3058#discussion_r821221192
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java
##########
@@ -1514,6 +1523,17 @@ public static void mvFileToFinalPath(Path specPath,
Configuration hconf,
fs.delete(taskTmpPath, true);
}
+ private static void createFileList(Set<FileStatus> filesKept, Path srcPath,
Path targetPath, FileSystem fs)
+ throws IOException {
+ try (FSDataOutputStream outStream = fs.create(new Path(targetPath,
BLOB_FILES_KEPT))) {
+ outStream.writeBytes(srcPath.toString() + System.lineSeparator());
Review comment:
Is it needed to write the srcPath as first entry? Only file paths can
be written out and during reading, first entry's parent directory can be
assumed as srcPath?
--
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]
