This is an automated email from the ASF dual-hosted git repository.

crazyhzm pushed a commit to branch 3.2
in repository https://gitbox.apache.org/repos/asf/dubbo.git


The following commit(s) were added to refs/heads/3.2 by this push:
     new 36b2ec267d Feature/dubbo3.2 rest remove first match break (#12837)
36b2ec267d is described below

commit 36b2ec267d87feda48998654252987469f81d8ef
Author: suncairong163 <[email protected]>
AuthorDate: Thu Aug 3 20:43:07 2023 +0800

    Feature/dubbo3.2 rest remove first match break (#12837)
    
    * support reExport
    
    * add rest method annotation support judge
    
    ---------
    
    Co-authored-by: suncr <[email protected]>
---
 .../dubbo/metadata/rest/AbstractServiceRestMetadataResolver.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/dubbo-metadata/dubbo-metadata-api/src/main/java/org/apache/dubbo/metadata/rest/AbstractServiceRestMetadataResolver.java
 
b/dubbo-metadata/dubbo-metadata-api/src/main/java/org/apache/dubbo/metadata/rest/AbstractServiceRestMetadataResolver.java
index ed00284b17..ec391d9483 100644
--- 
a/dubbo-metadata/dubbo-metadata-api/src/main/java/org/apache/dubbo/metadata/rest/AbstractServiceRestMetadataResolver.java
+++ 
b/dubbo-metadata/dubbo-metadata-api/src/main/java/org/apache/dubbo/metadata/rest/AbstractServiceRestMetadataResolver.java
@@ -198,8 +198,9 @@ public abstract class AbstractServiceRestMetadataResolver 
implements ServiceRest
             for (Method serviceMethod : serviceMethods) {
                 if (overrides(serviceMethod, declaredServiceMethod)) {
                     serviceMethodsMap.put(serviceMethod, 
declaredServiceMethod);
-                    // once method match ,break for decrease loop  times
-                    break;
+                    // override method count > 1
+//                    // once method match ,break for decrease loop  times
+//                    break;
                 }
             }
         }

Reply via email to