[pinpoint-apm#9633] Polishing

Author: emeroad

bootstraps/bootstrap-core/src/main/java/com/navercorp/pinpoint/bootstrap/interceptor/SpanRecursiveAroundInterceptor.java

@@ -176,10 +176,9 @@ private boolean initScope(final Trace trace) {
     }
 
     private void entryScope(final Trace trace) {
-        if (ScopeUtils.entryScope(trace, this.scopeName)) {
-            if (isDebug) {
-                logger.debug("Try enter trace scope={}", scopeName);
-            }
+        ScopeUtils.entryScope(trace, this.scopeName);
+        if (isDebug) {
+            logger.debug("Try enter trace scope={}", scopeName);
         }
     }
 

bootstraps/bootstrap-core/src/main/java/com/navercorp/pinpoint/bootstrap/util/ScopeUtils.java

@@ -11,17 +11,15 @@ public final class ScopeUtils {
     private ScopeUtils() {
     }
 
-    public static boolean entryAsyncTraceScope(final Trace trace) {
-        return entryScope(trace, ASYNC_TRACE_SCOPE);
+    public static void entryAsyncTraceScope(final Trace trace) {
+        entryScope(trace, ASYNC_TRACE_SCOPE);
     }
 
-    public static boolean entryScope(final Trace trace, final String scopeName) {
+    public static void entryScope(final Trace trace, final String scopeName) {
         final TraceScope scope = trace.getScope(scopeName);
         if (scope != null) {
             scope.tryEnter();
-            return true;
         }
-        return false;
     }
 
     public static boolean leaveAsyncTraceScope(final Trace trace) {

plugins/rocketmq/src/main/java/com/navercorp/pinpoint/plugin/rocketmq/interceptor/ProducerSendInterceptor.java

@@ -248,10 +248,9 @@ private boolean initScope(final Trace trace) {
     }
 
     private void entryScope(final Trace trace) {
-        if (ScopeUtils.entryScope(trace, SCOPE_NAME)) {
-            if (isDebug) {
-                logger.debug("Try enter trace scope={}", SCOPE_NAME);
-            }
+        ScopeUtils.entryScope(trace, SCOPE_NAME);
+        if (isDebug) {
+            logger.debug("Try enter trace scope={}", SCOPE_NAME);
         }
     }
 

plugins/thrift/src/main/java/com/navercorp/pinpoint/plugin/thrift/interceptor/tprotocol/server/TProtocolReadMessageEndInterceptor.java

@@ -379,10 +379,9 @@ private boolean initScope(final Trace trace) {
     }
 
     private void entryScope(final Trace trace) {
-        if (ScopeUtils.entryScope(trace, SCOPE_NAME)) {
-            if (isDebug) {
-                logger.debug("Try enter trace scope={}", SCOPE_NAME);
-            }
+        ScopeUtils.entryScope(trace, SCOPE_NAME);
+        if (isDebug) {
+            logger.debug("Try enter trace scope={}", SCOPE_NAME);
         }
     }
 

plugins/user/src/main/java/com/navercorp/pinpoint/plugin/user/interceptor/UserIncludeMethodInterceptor.java

@@ -181,10 +181,9 @@ private void deleteUserIncludeTrace(final Trace trace) {
     }
 
     private void entryUserIncludeTraceScope(final Trace trace) {
-        if (ScopeUtils.entryScope(trace, SCOPE_NAME)) {
-            if (isDebug) {
-                logger.debug("Try enter trace scope={}", SCOPE_NAME);
-            }
+        ScopeUtils.entryScope(trace, SCOPE_NAME);
+        if (isDebug) {
+            logger.debug("Try enter trace scope={}", SCOPE_NAME);
         }
     }
 

plugins/vertx/src/main/java/com/navercorp/pinpoint/plugin/vertx/interceptor/ServerConnectionHandleRequestInterceptor.java

@@ -289,10 +289,9 @@ private boolean initScope(final Trace trace) {
     }
 
     private void entryScope(final Trace trace) {
-        if (ScopeUtils.entryScope(trace, SCOPE_NAME)) {
-            if (isDebug) {
-                logger.debug("Try enter trace scope={}", SCOPE_NAME);
-            }
+        ScopeUtils.entryScope(trace, SCOPE_NAME);
+        if (isDebug) {
+            logger.debug("Try enter trace scope={}", SCOPE_NAME);
         }
     }
 

profiler/src/main/java/com/navercorp/pinpoint/profiler/context/id/LocalTraceRootImpl.java

@@ -44,9 +44,11 @@ public Shared getShared() {
 
     @Override
     public String toString() {
-        return "DefaultLocalTraceRoot{" +
-                ", agentId='" + agentId + '\'' +
+        return "LocalTraceRootImpl{" +
+                "agentId='" + agentId + '\'' +
+                ", localTransactionId=" + localTransactionId +
                 ", traceStartTime=" + traceStartTime +
+                ", shared=" + shared +
                 '}';
     }
 }

profiler/src/main/java/com/navercorp/pinpoint/profiler/context/id/RemoteTraceRootImpl.java

@@ -39,10 +39,12 @@ public TraceId getTraceId() {
 
     @Override
     public String toString() {
-        return "DefaultRemoteTraceRoot{" +
+        return "RemoteTraceRootImpl{" +
                 "traceId=" + traceId +
                 ", agentId='" + agentId + '\'' +
+                ", localTransactionId=" + localTransactionId +
                 ", traceStartTime=" + traceStartTime +
+                ", shared=" + shared +
                 '}';
     }