[GitHub] [ozone] neils-dev commented on pull request #4246: HDDS-7828. Make Ozone fs rm symbolic links command support posix behaviour

["neils-dev (via GitHub)" <gi...@apache.org>]

neils-dev commented on PR #4246:
URL: https://github.com/apache/ozone/pull/4246#issuecomment-1420152904

   Thanks @adoroszlai for your comment and for reviewing this.  
   
   >  I wonder if we can avoid duplicating the Delete handler from Hadoop by fixing the symlink argument being passed to FileStatusAdapter
   
   Unfortunately, supporting filestatus symlink does not let us avoid binding in an Ozone Delete handler. The handler is used to set the delete behavior for` -rm` (rm symlink) and `-rm` with trailing slash (to remove contents of symlink dir, through the call to the `FileSystem` `delete` with trailing slash).
    
   I too wanted to enable symlink in the filestatus, currently `null,` however it wasn't directly possible due to `FileStatus` on create only allowing an item to be either a directory or a symlink (asserts if both).  This causes problems with the `PathData` that expects directories for recursing paths, used in `listStatus` and other commands.


-- 
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: issues-unsubscribe@ozone.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org