Optimizing the type pool of byte-buddy (fix #11460)

Author: kylixs

apm-sniffer/apm-agent/src/main/java/org/apache/skywalking/apm/agent/SkyWalkingAgent.java

@@ -23,6 +23,8 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
+
 import net.bytebuddy.ByteBuddy;
 import net.bytebuddy.agent.builder.AgentBuilder;
 import net.bytebuddy.agent.builder.SWAgentBuilderDefault;
@@ -158,7 +160,8 @@ private static AgentBuilder newAgentBuilder() {
 
         SWNativeMethodStrategy nativeMethodStrategy = new SWNativeMethodStrategy(NAME_TRAIT);
         return new SWAgentBuilderDefault(byteBuddy, nativeMethodStrategy)
-                .with(AgentBuilder.DescriptionStrategy.Default.POOL_FIRST);
+                .with(AgentBuilder.DescriptionStrategy.Default.POOL_FIRST)
+                .with(new AgentBuilder.PoolStrategy.WithTypePoolCache.Simple(new ConcurrentHashMap<>()));
     }
 
     private static class Transformer implements AgentBuilder.Transformer {

apm-sniffer/bytebuddy-patch/src/test/java/org/apache/skywalking/apm/agent/bytebuddy/cases/AbstractInterceptTest.java

@@ -51,6 +51,7 @@
 import java.security.ProtectionDomain;
 import java.util.Arrays;
 import java.util.List;
+import java.util.concurrent.ConcurrentHashMap;
 
 public class AbstractInterceptTest {
     public static final String BIZ_FOO_CLASS_NAME = "org.apache.skywalking.apm.agent.bytebuddy.biz.BizFoo";
@@ -174,6 +175,7 @@ protected AgentBuilder newAgentBuilder(String nameTrait) {
         return new SWAgentBuilderDefault(byteBuddy, new SWNativeMethodStrategy(nameTrait))
                 .with(AgentBuilder.RedefinitionStrategy.RETRANSFORMATION)
                 .with(AgentBuilder.DescriptionStrategy.Default.POOL_FIRST)
+                .with(new AgentBuilder.PoolStrategy.WithTypePoolCache.Simple(new ConcurrentHashMap<>()))
                 .with(new SWClassFileLocator(ByteBuddyAgent.install(), getClassLoader()));
     }
 

apm-sniffer/bytebuddy-patch/src/test/java/org/apache/skywalking/apm/agent/bytebuddy/cases/ReTransform2Test.java

@@ -54,7 +54,11 @@ public void testInterceptConstructor() throws Exception {
         checkMethodInterceptor(SAY_HELLO_METHOD, 2);
 
         // do retransform
-        reTransform(instrumentation, BizFoo.class);
+        int retransformCount = 30;
+        for (int i = 0; i < retransformCount; i++) {
+            reTransform(instrumentation, BizFoo.class);
+            // reTransform(instrumentation, ProjectService.class);
+        }
 
         // test again
         callBizFoo(2);