[GitHub] [pinot] Jackie-Jiang commented on pull request #8200: add copy recursive API to pinotFS

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on pull request #8200:
URL: https://github.com/apache/pinot/pull/8200#issuecomment-1039626470


   @mcvsubbu Trying to understand the the problem we are trying to solve in #8162. IIUC, we want to have an API in `PinotFS` that copies only file to another location, but reject the copy request if src is a directory. I'd suggest still keeping the API for directory copy because that is a very useful operation for a FS. It is caller's responsibility to call the correct API, but `PinotFS` should provide both functionalities. (Actually copy only file but not directory is not mandatory because caller can call `isDirectory()` first and reject the request when the src is a directory).


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org