wu-sheng commented on a change in pull request #3761: Rabbitmq instrument
consumer class correctly to get duration reported…
URL: https://github.com/apache/skywalking/pull/3761#discussion_r342111496
##########
File path:
apm-sniffer/apm-agent-core/src/main/java/org/apache/skywalking/apm/agent/core/plugin/match/HierarchyMatch.java
##########
@@ -46,6 +47,11 @@ private HierarchyMatch(String[] parentTypes) {
this.parentTypes = parentTypes;
}
+ private HierarchyMatch(String[] parentTypes, boolean interfaceOnlyMatch) {
+ this(parentTypes);
+ this.interfaceOnlyMatch = interfaceOnlyMatch;
+ }
+
Review comment:
Replace `InstanceMethodsInterceptPoint` to
`DeclaredInstanceMethodsInterceptPoint` in your instrumentation define. It is
an easy change. They you could test again. Then in the hierarchy case, it
should workw as your exepected. @viswaramamoorthy
This interceptor point mean only this method is being implemented in this
class, rather that existing the parent class, the intercept happens.
I think you are closing the right one.
----------------------------------------------------------------
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
