Improve benchmark and caching optimizations.

Author: tylerbenson

dd-java-agent/benchmark/benchmark.gradle

@@ -60,13 +60,12 @@ task jmhAgent(type: JavaExec, dependsOn: project.tasks.jmhCompileGeneratedClasse
   main = "org.openjdk.jmh.Main"
   args += ["-tu", "us"]
   args += ["-bm", "avgt"]
-  args += ["-prof", "stack"]
-//  args += ["-r", "5s"]
-//  args += ["-i", "5"]
+//  args += ["-prof", "stack:lines=5;detailLine=true;period=5;excludePackages=true"]
   args += ["-f", "1"]
   args += ["-foe", "true"]
-//  args += ["-w", "2s"]
+
 //  args += ["-wi", "2"]
+//  args += ["-i", "5"]
 }
 
 tasks.jmhAgent.dependsOn project(':dd-java-agent').shadowJar

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/ClassRetransformingBenchmark.java

@@ -1,7 +1,7 @@
 package com.datadoghq.benchmark;
 
-import com.datadoghq.benchmark.classes.DeepClass;
-import com.datadoghq.benchmark.classes.SimpleClass;
+import com.datadoghq.benchmark.classes.TracedClass;
+import com.datadoghq.benchmark.classes.UntracedClass;
 import java.lang.instrument.Instrumentation;
 import java.lang.instrument.UnmodifiableClassException;
 import java.lang.reflect.Field;
@@ -54,13 +54,14 @@ public void stopAgent() {
   }
 
   @Benchmark
-  public void testSimpleRetransform(final BenchmarkState state) throws UnmodifiableClassException {
-    state.inst.retransformClasses(SimpleClass.class);
+  public void testUntracedRetransform(final BenchmarkState state)
+      throws UnmodifiableClassException {
+    state.inst.retransformClasses(UntracedClass.class);
   }
 
   @Benchmark
-  public void testDeepRetransform(final BenchmarkState state) throws UnmodifiableClassException {
-    state.inst.retransformClasses(DeepClass.class);
+  public void testTracedRetransform(final BenchmarkState state) throws UnmodifiableClassException {
+    state.inst.retransformClasses(TracedClass.class);
   }
 
   @Fork(jvmArgsAppend = "-javaagent:releases/dd-java-agent-0.2.2.jar")

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/classes/A.java

@@ -1,8 +1,5 @@
 package com.datadoghq.benchmark.classes;
 
-import com.datadoghq.trace.Trace;
-
 public interface A {
-  @Trace
-  void interfaceTrace();
+  void a();
 }

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/classes/B.java

@@ -1,8 +1,5 @@
 package com.datadoghq.benchmark.classes;
 
-import com.datadoghq.trace.Trace;
-
 public interface B extends A {
-  @Trace
-  void something();
+  void b();
 }

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/classes/C.java

@@ -1,5 +1,5 @@
 package com.datadoghq.benchmark.classes;
 
 public interface C extends B {
-  void somethingElse();
+  void c();
 }

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/classes/D.java

@@ -0,0 +1,5 @@
+package com.datadoghq.benchmark.classes;
+
+public interface D extends C {
+  void d();
+}

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/classes/DeepClass.java

@@ -0,0 +1,5 @@
+package com.datadoghq.benchmark.classes;
+
+public interface E extends D {
+  void e();
+}

dd-java-agent/benchmark/src/jmh/java/com/datadoghq/benchmark/classes/E.java

@@ -0,0 +1,7 @@
+package com.datadoghq.benchmark.classes;
+
+public abstract class F implements E {
+  public abstract void f();
+
+  public void g() {}
+}