Reuse ExecutionStrategyInstrumentationContext.NOOP in DataLoaderDispatcherInstrumentation (#3068)

dfa1 · web-flow · commit 86d38c1a3518 · 2023-01-30T12:04:00.000+11:00
diff --git a/src/main/java/graphql/execution/instrumentation/dataloader/DataLoaderDispatcherInstrumentation.java b/src/main/java/graphql/execution/instrumentation/dataloader/DataLoaderDispatcherInstrumentation.java
@@ -129,16 +129,7 @@ private boolean isDataLoaderCompatibleExecution(ExecutionContext executionContex
         // if there are no data loaders, there is nothing to do
         //
         if (state.hasNoDataLoaders()) {
-            return new ExecutionStrategyInstrumentationContext() {
-                @Override
-                public void onDispatched(CompletableFuture<ExecutionResult> result) {
-                }
-
-                @Override
-                public void onCompleted(ExecutionResult result, Throwable t) {
-                }
-            };
-
+            return ExecutionStrategyInstrumentationContext.NOOP;
         }
         return state.getApproach().beginExecutionStrategy(parameters, state.getState());
     }
