sunithabeeram commented on a change in pull request #3834: Unify move method in
PinotFS
URL: https://github.com/apache/incubator-pinot/pull/3834#discussion_r258154167
##########
File path:
pinot-filesystem/src/main/java/org/apache/pinot/filesystem/PinotFS.java
##########
@@ -59,12 +59,15 @@ public abstract boolean delete(URI segmentUri, boolean
forceDelete)
* Moves the file or directory from the src to dst. Does not keep the
original file. If the dst has parent directories
* that haven't been created, this method will create all the necessary
parent directories.
* If both src and dst are files, dst will be overwritten.
- * If src is a file and dst is a directory, src file will get moved under
dst directory.
+ * If src is a file and dst is an empty directory, src file will get moved
under dst directory.
* If both src and dst are directories, src directory will get moved under
dst directory.
* If src is a directory and dst is a file, operation will fail.
* For example, if a file /a/b/c is moved to a file /x/y/z, in the case of
overwrite, the directory /a/b still exists,
* but will not contain the file 'c'. Instead, /x/y/z will contain the
contents of 'c'.
* If a file /a is moved to a directory /x/y, all the original files under
/x/y will be kept.
+ *
+ * As of now, the usage of this API in Pinot is just to move a file to a
file. So please specify the full names of
+ * both src and dst to make less confusion.
Review comment:
Remove "to make less confusion"
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
