omalley commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r819950355
########## 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: It still should use the "." as hierarchy and "-" between words. I guess closer would be: fs.viewfs.trash.force-inside-mount-point -- 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