[SCB-2120]fix operation override when using schema interface

demo/demo-jaxrs/jaxrs-server/src/main/java/org/apache/servicecomb/demo/jaxrs/server/SchemeInterfaceJaxrsImpl.java

@@ -31,6 +31,10 @@ public int reduce(int a, int b) {
     return a - b;
   }
 
+  public int add(String a, String b) {
+    return Integer.valueOf(a) + Integer.valueOf(b);
+  }
+
   @Override
   public Page<String> interfaceModel(Page<String> model) {
     return model;

foundations/foundation-protobuf/src/main/java/org/apache/servicecomb/foundation/protobuf/internal/schema/deserializer/MessageReadSchema.java

@@ -164,12 +164,14 @@ public void mergeFrom(InputEx input, T message) throws IOException {
         n = input.readFieldNumber();
       }
     } catch (Throwable e) {
-      Field protoField = fieldSchema.getProtoField();
-      LOGGER.error("Failed to mergeFrom, field={}:{}, type={}",
-          protoField.getType().getCanonicalName(),
-          protoField.getName(),
-          protoField.getTypeName(),
-          e.getMessage());
+      if (fieldSchema != null) {
+        Field protoField = fieldSchema.getProtoField();
+        LOGGER.error("Failed to mergeFrom, field={}:{}, type={}",
+            protoField.getType().getCanonicalName(),
+            protoField.getName(),
+            protoField.getTypeName(),
+            e.getMessage());
+      }
       throw e;
     }
   }

swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/engine/SwaggerEnvironment.java

@@ -104,7 +104,9 @@ public SwaggerProducer createProducer(Object producerInstance, Swagger swagger)
   }
 
   public SwaggerProducer createProducer(Object producerInstance, Class<?> schemaInterface, Swagger swagger) {
-    swagger = checkAndGenerateSwagger(producerInstance, schemaInterface, swagger);
+    Class<?> producerCls = targetSwaggerClass(producerInstance, schemaInterface);
+
+    swagger = checkAndGenerateSwagger(producerCls, swagger);
 
     Map<Class<?>, ContextArgumentMapperFactory> contextFactories = SPIServiceUtils
         .getOrLoadSortedService(ProducerContextArgumentMapperFactory.class)
@@ -115,7 +117,6 @@ public SwaggerProducer createProducer(Object producerInstance, Class<?> schemaIn
 
     SwaggerOperations swaggerOperations = new SwaggerOperations(swagger);
 
-    Class<?> producerCls = BeanUtils.getImplClassFromBean(producerInstance);
     Map<String, Method> visibleProducerMethods = MethodUtils.findSwaggerMethodsMapOfOperationId(producerCls);
 
     SwaggerProducer producer = new SwaggerProducer();
@@ -159,15 +160,17 @@ public SwaggerProducer createProducer(Object producerInstance, Class<?> schemaIn
     return producer;
   }
 
-  private Swagger checkAndGenerateSwagger(Object producerInstance, Class<?> schemaInterface, Swagger swagger) {
+  private Swagger checkAndGenerateSwagger(Class<?> swaggerClass, Swagger swagger) {
     if (swagger == null) {
-      if (schemaInterface != null && !Object.class.equals(schemaInterface)) {
-        swagger = SwaggerGenerator.generate(schemaInterface);
-      } else {
-        Class<?> producerCls = BeanUtils.getImplClassFromBean(producerInstance);
-        swagger = SwaggerGenerator.generate(producerCls);
-      }
+      swagger = SwaggerGenerator.generate(swaggerClass);
     }
     return swagger;
   }
+
+  private Class<?> targetSwaggerClass(Object producerInstance, Class<?> schemaInterface) {
+    if (schemaInterface != null && !Object.class.equals(schemaInterface)) {
+      return schemaInterface;
+    }
+    return BeanUtils.getImplClassFromBean(producerInstance);
+  }
 }