zhangxiping1 commented on code in PR #4269:
URL: https://github.com/apache/hadoop/pull/4269#discussion_r873283844
##########
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:
OK，In [HDFS-16024](https://issues.apache.org/jira/browse/HDFS-16024). The
trash path is resolved by removing the prefix of the trash path. I think in the
isPathAll() function, the prefix of the Trash path also needs to be processed.
The isPathAll method is used to determine whether any resolved remotelocation
is performed. Therefore, the isPathAll method should be consistent with the SRC
path (excluding the trash prefix). Because all remotelocation resolved
previously is SRC based.
--
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]
