zhangxiping1 commented on code in PR #4269:
URL: https://github.com/apache/hadoop/pull/4269#discussion_r873287952
##########
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RouterRpcServer.java:
##########
@@ -1896,6 +1897,9 @@ public FederationRPCMetrics getRPCMetrics() {
boolean isPathAll(final String path) {
if (subclusterResolver instanceof MountTableResolver) {
try {
+ if(isTrashPath(path)){
+ return true;
Review Comment:
I can make two changes in isPathAll fuction：
1. Process the Trash path, remove the prefix, and check
2. Check whether it is the trash path
If we delete, mkdir, LS on the recycle bin data, if reslove gets multiple
Remotelocation, then we should operate on all remotelocation, so I'm going to
choose the second option. But the first is certainly fine.
If you think there's something wrong, you can talk me out of it, thank you.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
