xinglin commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r819943782
########## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/viewfs/Constants.java ########## @@ -134,8 +134,10 @@ HCFSMountTableConfigLoader.class; /** - * Enable ViewFileSystem to return a trashRoot which is local to mount point. + * Enable ViewFileSystem to return a trashRoot which is in the root dir of a + * mount point. */ - String CONFIG_VIEWFS_MOUNT_POINT_LOCAL_TRASH = "fs.viewfs.mount.point.local.trash"; - boolean CONFIG_VIEWFS_MOUNT_POINT_LOCAL_TRASH_DEFAULT = false; + String CONFIG_VIEWFS_TRASH_ROOT_UNDER_MOUNT_POINT_ROOT = + "fs.viewfs.trash.root.under.mount.point.root"; Review comment: localized trash is not appropriate as the default hadoop implementation is already able to return a trash root that is within the mount point Filesystem. Thus, I renamed the new configuration. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org