harishjp commented on a change in pull request #2120:
URL: https://github.com/apache/hive/pull/2120#discussion_r635977406



##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java
##########
@@ -1093,40 +1093,69 @@ public static void rename(FileSystem fs, Path src, Path 
dst) throws IOException,
     }
   }
 
-  private static void moveFile(FileSystem fs, FileStatus file, Path dst) 
throws IOException,
+  private static void moveFileOrDir(FileSystem fs, FileStatus file, Path dst) 
throws IOException,

Review comment:
       Test renameOrMove files calls this, and is the public API. Tests for 
that should cover cases here.




-- 
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.

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