[ 
https://issues.apache.org/jira/browse/HDFS-17733?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17928982#comment-17928982
 ] 

ASF GitHub Bot commented on HDFS-17733:
---------------------------------------

hfutatzhanghb commented on code in PR #7415:
URL: https://github.com/apache/hadoop/pull/7415#discussion_r1964657428


##########
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/async/RouterAsyncClientProtocol.java:
##########
@@ -1057,33 +1057,33 @@ public boolean setReplication(String src, short 
replication)
    *         subclusters else false in all other scenarios.
    * @throws IOException if unable to get the file status.
    */
+  @VisibleForTesting
   @Override
   public boolean isMultiDestDirectory(String src) throws IOException {
-    try {
+    asyncComplete(false);

Review Comment:
   @KeeProMise very nice suggestion, thanks for reviewing. have fixed.





> [ARR] Optimize isMultiDestDirectory method using AsyncUtil class
> ----------------------------------------------------------------
>
>                 Key: HDFS-17733
>                 URL: https://issues.apache.org/jira/browse/HDFS-17733
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: rbf
>            Reporter: farmmamba
>            Assignee: farmmamba
>            Priority: Major
>              Labels: pull-request-available
>
> Optimize isMultiDestDirectory method using AsyncUtil class.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to