HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path#4034

Closed

xinglin wants to merge 26 commits intoapache:trunkfrom

Commits on Feb 25, 2022

Change to check for EZ/snapshots
Xing Lin
committed
Change getTrashRoot to return viewFS path, instead of targetFS path
Xing Lin
committed
Rename MOUNT_POINT_LOCAL_TRASH macro
Xing Lin
committed
Modified getTrashRoot/s to return hybrid paths.
Xing Lin
committed

Commits on Feb 26, 2022

Minor edits
Xing Lin
committed
More minor edits
Xing Lin
committed
HADOOP-18144. getTrashRoot/s in ViewFileSystem should return viewFS path, not targetFS path
Xing Lin
committed

Commits on Mar 5, 2022

check trashRoot from targetFS with mountpoint.targetPath.
Xing Lin
committed
Rename MACRO for our flag
Xing Lin
committed
Renamed flag to CONFIG_VIEWFS_TRASH_USE_VIEWFS_PATH
Xing Lin
committed
Alright, renamed flag back to CONFIG_VIEWFS_TRASH_FORCE_INSIDE_MOUNT_POINT
Xing Lin
committed
minor fix fo the warning
Xing Lin
committed
Fixed a unit test function name
Xing Lin
committed

Commits on Mar 7, 2022

changed CONFIG_VIEWFS_TRASH_FORCE_INSIDE_MOUNT_POINT_DEFAULT to false
Xing Lin
committed
Fixed checkstyle
Xing Lin
committed
Fixed out-of-date flag name in comments
Xing Lin
committed

Commits on Mar 8, 2022

Addressed Owen's comment and add a check for default trash when mount target is null or /.
Xing Lin
committed
fixed line break
Xing Lin
committed
Fixed the path comparison
Xing Lin
committed
fix spotbugs
Xing Lin
committed

Commits on Mar 11, 2022

Another pass to incorporate Owen's changes.
Xing Lin
committed

Commits on Mar 12, 2022

checkpoint of use if/else to decide expectedTrash.
Xing Lin
committed
Added and overwrite getTrashRootInFallBackFS()
Xing Lin
committed
Fix new lines
Xing Lin
committed
minor fix to comments.
Xing Lin
committed

Commits on Mar 14, 2022

GetTrashRoots(): include trash roots in fallback FS as well.
Xing Lin
committed