From 5fd5ee4e5b5c1c62da3e4f1da8e1d4a522e41017 Mon Sep 17 00:00:00 2001 From: dugenkui03 Date: Sat, 19 Mar 2022 01:39:25 +0800 Subject: [PATCH] Make NoOpStatisticsCollector to be default StatisticsCollector --- .../graphql/execution/DefaultBatchLoaderRegistry.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/spring-graphql/src/main/java/org/springframework/graphql/execution/DefaultBatchLoaderRegistry.java b/spring-graphql/src/main/java/org/springframework/graphql/execution/DefaultBatchLoaderRegistry.java index a5c868649..feea6f341 100644 --- a/spring-graphql/src/main/java/org/springframework/graphql/execution/DefaultBatchLoaderRegistry.java +++ b/spring-graphql/src/main/java/org/springframework/graphql/execution/DefaultBatchLoaderRegistry.java @@ -32,6 +32,7 @@ import org.dataloader.DataLoaderOptions; import org.dataloader.DataLoaderRegistry; import org.dataloader.MappedBatchLoaderWithContext; +import org.dataloader.stats.NoOpStatisticsCollector; import reactor.core.publisher.Flux; import reactor.core.publisher.Mono; import reactor.util.context.ContextView; @@ -68,7 +69,8 @@ public RegistrationSpec forName(String name) { @Override public void registerDataLoaders(DataLoaderRegistry registry, GraphQLContext context) { BatchLoaderContextProvider contextProvider = () -> context; - DataLoaderOptions defaultOptions = DataLoaderOptions.newOptions().setBatchLoaderContextProvider(contextProvider); + DataLoaderOptions defaultOptions = DataLoaderOptions.newOptions().setBatchLoaderContextProvider(contextProvider) + .setStatisticsCollector(NoOpStatisticsCollector::new); for (ReactorBatchLoader loader : this.loaders) { DataLoaderOptions options = loader.getOptionsOrDefault(contextProvider, defaultOptions); DataLoader dataLoader = DataLoaderFactory.newDataLoader(loader, options); @@ -117,7 +119,9 @@ public RegistrationSpec withName(String name) { @Override public RegistrationSpec withOptions(Consumer optionsConsumer) { - this.options = (this.options != null ? this.options : DataLoaderOptions.newOptions()); + this.options = (this.options != null ? + this.options : + DataLoaderOptions.newOptions().setStatisticsCollector(NoOpStatisticsCollector::new)); optionsConsumer.accept(this.options); return this; }