wu-sheng commented on a change in pull request #2710: Remove the ex which not 
thrown. & add logs for interceptor.
URL: https://github.com/apache/skywalking/pull/2710#discussion_r285413201
 
 

 ##########
 File path: 
apm-sniffer/apm-agent-core/src/main/java/org/apache/skywalking/apm/agent/core/plugin/loader/InterceptorInstanceLoader.java
 ##########
 @@ -80,6 +78,9 @@
             }
             if (inst != null) {
                 INSTANCE_CACHE.put(instanceKey, inst);
+                logger.debug("interceptor instance is created for class '{}', 
instanceKey is: {}", className, instanceKey);
 
 Review comment:
   I don't know whether we need that. I am OK to discuss. Right now, there are 
only @ascrutae @candyleer and I have experiences and developed core codes. The 
opinion of mine is, most people even don't understand what the agent really 
does.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to