more performance, disable tests

andimarek · andimarek · commit dd932bf250dd · 2025-10-12T20:22:29.000+10:00
diff --git a/src/jmh/java/performance/DataLoaderPerformance.java b/src/jmh/java/performance/DataLoaderPerformance.java
@@ -36,7 +36,7 @@
 @State(Scope.Benchmark)
 @Warmup(iterations = 2, time = 5)
 @Measurement(iterations = 5)
-@Fork(3)
+@Fork(2)
 public class DataLoaderPerformance {
 
     static Owner o1 = new Owner("O-1", "Andi", List.of("P-1", "P-2", "P-3"));
diff --git a/src/main/java/graphql/schema/DataLoaderWithContext.java b/src/main/java/graphql/schema/DataLoaderWithContext.java
@@ -10,7 +10,6 @@
 import org.jspecify.annotations.NullMarked;
 import org.jspecify.annotations.Nullable;
 
-import java.util.List;
 import java.util.concurrent.CompletableFuture;
 
 @Internal
@@ -43,16 +42,4 @@ public CompletableFuture<V> load(@NonNull K key, @Nullable Object keyContext) {
         }
         return result;
     }
-
-    @Override
-    public CompletableFuture<List<V>> dispatch() {
-        CompletableFuture<List<V>> dispatchResult = delegate.dispatch();
-        dispatchResult.whenComplete((result, error) -> {
-            if (result != null && result.size() > 0) {
-                DataFetchingEnvironmentImpl.DFEInternalState dfeInternalState = (DataFetchingEnvironmentImpl.DFEInternalState) dfe.toInternal();
-                dfeInternalState.getProfiler().manualDispatch(dataLoaderName, dfe.getExecutionStepInfo().getPath().getLevel(), result.size());
-            }
-        });
-        return dispatchResult;
-    }
 }
diff --git a/src/test/groovy/graphql/ProfilerTest.groovy b/src/test/groovy/graphql/ProfilerTest.groovy
@@ -14,6 +14,7 @@ import org.dataloader.BatchLoader
 import org.dataloader.DataLoader
 import org.dataloader.DataLoaderFactory
 import org.dataloader.DataLoaderRegistry
+import spock.lang.Ignore
 import spock.lang.Specification
 
 import java.time.Duration
@@ -176,6 +177,7 @@ class ProfilerTest extends Specification {
     }
 
 
+    @Ignore("not available for performance reasons at the moment")
     def "manual dataloader dispatch"() {
         given:
         def sdl = '''
@@ -230,6 +232,7 @@ class ProfilerTest extends Specification {
 
     }
 
+    @Ignore("not available for performance reasons at the moment")
     def "cached dataloader values"() {
         given:
         def sdl = '''
@@ -517,6 +520,7 @@ class ProfilerTest extends Specification {
 
     }
 
+    @Ignore("not available for performance reasons at the moment")
     def "dataloader usage"() {
         given:
         def sdl = '''
