[GitHub] [hadoop] xinglin commented on a change in pull request #4034: HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path

2022-03-09 Thread GitBox
xinglin commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r823155360 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/viewfs/ViewFileSystem.java ## @@ -1128,51 +1128,58 @@ public BlockSto

[GitHub] [hadoop] xinglin commented on a change in pull request #4034: HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path

2022-03-04 Thread GitBox
xinglin commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r820035459 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/TrashPolicyDefault.java ## @@ -191,8 +191,8 @@ public boolean moveToT

[GitHub] [hadoop] xinglin commented on a change in pull request #4034: HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path

2022-03-04 Thread GitBox
xinglin commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r820037169 ## File path: hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/viewfs/ViewFileSystemBaseTest.java ## @@ -1141,60 +1143,52 @@ public

[GitHub] [hadoop] xinglin commented on a change in pull request #4034: HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path

2022-03-04 Thread GitBox
xinglin commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r820035684 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/TrashPolicyDefault.java ## @@ -191,8 +191,8 @@ public boolean moveToT

[GitHub] [hadoop] xinglin commented on a change in pull request #4034: HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path

2022-03-04 Thread GitBox
xinglin commented on a change in pull request #4034: URL: https://github.com/apache/hadoop/pull/4034#discussion_r820035459 ## File path: hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/TrashPolicyDefault.java ## @@ -191,8 +191,8 @@ public boolean moveToT

[GitHub] [hadoop] xinglin commented on a change in pull request #4034: HADOOP-18144. getTrashRoot/s in ViewFileSystem return a viewFS path

2022-03-04 Thread GitBox
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 @@ HCFSMountTableCo