Note enhancement points

huicunjun · huicunjun · commit 303ffd42554d · 2025-07-27T10:53:17.000+08:00
diff --git a/apm-sniffer/apm-sdk-plugin/grpc-1.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/grpc/v1/define/AbstractServerImplBuilderInstrumentation.java b/apm-sniffer/apm-sdk-plugin/grpc-1.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/grpc/v1/define/AbstractServerImplBuilderInstrumentation.java
@@ -64,8 +64,9 @@ public boolean isOverrideArgs() {
     @Override
     protected ClassMatch enhanceClass() {
         return MultiClassNameMatch.byMultiClassMatch(
-                "io.grpc.internal.AbstractServerImplBuilder",
-                "io.grpc.internal.ServerImplBuilder",
-                "io.grpc.internal.ForwardingServerBuilder");
+                "io.grpc.internal.AbstractServerImplBuilder", //grpc version <= 1.58.1
+                "io.grpc.internal.ServerImplBuilder", //grpc version >= 1.59.0
+                "io.grpc.internal.ForwardingServerBuilder" //grpc version >= 1.59.0
+        );
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -64,8 +64,9 @@ public boolean isOverrideArgs() {`
`64`	`64`	`@Override`
`65`	`65`	`protected ClassMatch enhanceClass() {`
`66`	`66`	`return MultiClassNameMatch.byMultiClassMatch(`
`67`		`- "io.grpc.internal.AbstractServerImplBuilder",`
`68`		`- "io.grpc.internal.ServerImplBuilder",`
`69`		`- "io.grpc.internal.ForwardingServerBuilder");`
	`67`	`+ "io.grpc.internal.AbstractServerImplBuilder", //grpc version <= 1.58.1`
	`68`	`+ "io.grpc.internal.ServerImplBuilder", //grpc version >= 1.59.0`
	`69`	`+ "io.grpc.internal.ForwardingServerBuilder" //grpc version >= 1.59.0`
	`70`	`+ );`
`70`	`71`	`}`
`71`	`72`	`}`