Resolve review comments

integration-tx-api/src/main/java/org/apache/seata/integration/tx/api/interceptor/InvocationHandlerType.java

@@ -0,0 +1,26 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.seata.integration.tx.api.interceptor;
+
+/**
+ * The  InvocationHandlerType enum
+ */
+public enum InvocationHandlerType {
+
+    GlobalTransactional, TwoPhaseAnnotation
+
+}

integration-tx-api/src/main/java/org/apache/seata/integration/tx/api/interceptor/handler/GlobalTransactionalInterceptorHandler.java

@@ -41,6 +41,7 @@
 import org.apache.seata.core.model.GlobalLockConfig;
 import org.apache.seata.integration.tx.api.annotation.AspectTransactional;
 import org.apache.seata.integration.tx.api.event.DegradeCheckEvent;
+import org.apache.seata.integration.tx.api.interceptor.InvocationHandlerType;
 import org.apache.seata.integration.tx.api.interceptor.InvocationWrapper;
 import org.apache.seata.integration.tx.api.interceptor.SeataInterceptorPosition;
 import org.apache.seata.integration.tx.api.util.ClassUtils;
@@ -413,6 +414,6 @@ public SeataInterceptorPosition getPosition() {
 
     @Override
     public String type() {
-        return "GlobalTransactional";
+        return InvocationHandlerType.GlobalTransactional.name();
     }
 }

integration-tx-api/src/main/java/org/apache/seata/integration/tx/api/interceptor/parser/DefaultInterfaceParser.java

@@ -84,19 +84,19 @@ public ProxyInvocationHandler parserInterfaceToProxy(Object target, String objec
 
         Collections.sort(invocationHandlerList, Comparator.comparingInt(ProxyInvocationHandler::order));
 
-        ProxyInvocationHandler result = null;
+        ProxyInvocationHandler first = null;
         ProxyInvocationHandler last = null;
         for (ProxyInvocationHandler proxyInvocationHandler : invocationHandlerList) {
-            if (result == null) {
-                result = proxyInvocationHandler;
+            if (first == null) {
+                first = proxyInvocationHandler;
             }
             if (last != null) {
                 last.setNextProxyInvocationHandler(proxyInvocationHandler);
             }
             last = proxyInvocationHandler;
         }
 
-        return result;
+        return first;
     }
 
     @Override

tcc/src/main/java/org/apache/seata/rm/tcc/interceptor/TccActionInterceptorHandler.java

@@ -30,6 +30,7 @@
 import org.apache.seata.core.model.BranchType;
 import org.apache.seata.integration.tx.api.fence.config.CommonFenceConfig;
 import org.apache.seata.integration.tx.api.interceptor.ActionInterceptorHandler;
+import org.apache.seata.integration.tx.api.interceptor.InvocationHandlerType;
 import org.apache.seata.integration.tx.api.interceptor.InvocationWrapper;
 import org.apache.seata.integration.tx.api.interceptor.SeataInterceptorPosition;
 import org.apache.seata.integration.tx.api.interceptor.TwoPhaseBusinessActionParam;
@@ -171,6 +172,6 @@ public int order() {
 
     @Override
     public String type() {
-        return "twoPhaseAnnotation";
+        return InvocationHandlerType.TwoPhaseAnnotation.name();
     }
 }