Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Commit 159d5b7

Browse files
authored
Merge pull request #951 from bsara/issue-950-fix-metrics-instrumentation
2 parents cb82525 + 4c5a3e0 commit 159d5b7

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

graphql-spring-boot-autoconfigure/src/main/java/graphql/kickstart/autoconfigure/web/servlet/metrics/MetricsInstrumentation.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package graphql.kickstart.autoconfigure.web.servlet.metrics;
22

33
import graphql.ExecutionResult;
4+
import graphql.execution.instrumentation.InstrumentationState;
45
import graphql.execution.instrumentation.parameters.InstrumentationExecutionParameters;
56
import graphql.execution.instrumentation.tracing.TracingInstrumentation;
67
import io.micrometer.core.instrument.MeterRegistry;
@@ -39,6 +40,12 @@ public MetricsInstrumentation(MeterRegistry meterRegistry, boolean tracingEnable
3940
@Override
4041
public CompletableFuture<ExecutionResult> instrumentExecutionResult(
4142
ExecutionResult executionResult, InstrumentationExecutionParameters parameters) {
43+
return this.instrumentExecutionResult(executionResult, parameters, null);
44+
}
45+
46+
@Override
47+
public CompletableFuture<ExecutionResult> instrumentExecutionResult(
48+
ExecutionResult executionResult, InstrumentationExecutionParameters parameters, InstrumentationState rawState) {
4249

4350
if (executionResult.getExtensions() != null
4451
&& executionResult.getExtensions().containsKey(TRACING)) {

0 commit comments

Comments
 (0)