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

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

commit 0ad3577dd40b535e38d7262a544d1f705015edab
Author: beiwei.ly <[email protected]>
AuthorDate: Tue Oct 29 16:33:42 2019 +0800

    enhance generic invocation check
---
 .../src/main/java/org/apache/dubbo/rpc/support/RpcUtils.java         | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/support/RpcUtils.java
 
b/dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/support/RpcUtils.java
index 94faa57..7e0da8f 100644
--- 
a/dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/support/RpcUtils.java
+++ 
b/dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/support/RpcUtils.java
@@ -24,6 +24,7 @@ import org.apache.dubbo.common.utils.StringUtils;
 import org.apache.dubbo.rpc.Invocation;
 import org.apache.dubbo.rpc.InvokeMode;
 import org.apache.dubbo.rpc.RpcInvocation;
+import org.apache.dubbo.rpc.service.GenericService;
 
 import java.lang.reflect.Method;
 import java.lang.reflect.Type;
@@ -48,6 +49,7 @@ public class RpcUtils {
         try {
             if (invocation != null && invocation.getInvoker() != null
                     && invocation.getInvoker().getUrl() != null
+                    && invocation.getInvoker().getInterface() != 
GenericService.class
                     && !invocation.getMethodName().startsWith("$")) {
                 String service = 
invocation.getInvoker().getUrl().getServiceInterface();
                 if (StringUtils.isNotEmpty(service)) {
@@ -62,7 +64,7 @@ public class RpcUtils {
                 }
             }
         } catch (Throwable t) {
-            // ignore
+            logger.warn(t.getMessage(), t);
         }
         return null;
     }
@@ -71,6 +73,7 @@ public class RpcUtils {
         try {
             if (invocation != null && invocation.getInvoker() != null
                     && invocation.getInvoker().getUrl() != null
+                    && invocation.getInvoker().getInterface() != 
GenericService.class
                     && !invocation.getMethodName().startsWith("$")) {
                 String service = 
invocation.getInvoker().getUrl().getServiceInterface();
                 if (StringUtils.isNotEmpty(service)) {

Reply via email to