Skip to content

Commit 5030cfe

Browse files
committed
Revert "Make MeterRegistry lazy in TimerObservationHandlerObservationRegistryCustomizer"
This reverts commit 0439127.
1 parent e35140d commit 5030cfe

File tree

3 files changed

+5
-13
lines changed

3 files changed

+5
-13
lines changed

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/observation/ObservationAutoConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ static class MetricsConfiguration {
6767

6868
@Bean
6969
TimerObservationHandlerObservationRegistryCustomizer enableTimerObservationHandler(
70-
ObjectProvider<MeterRegistry> meterRegistry) {
70+
MeterRegistry meterRegistry) {
7171
return new TimerObservationHandlerObservationRegistryCustomizer(meterRegistry);
7272
}
7373

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/observation/TimerObservationHandlerObservationRegistryCustomizer.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@
2020
import io.micrometer.core.instrument.observation.TimerObservationHandler;
2121
import io.micrometer.observation.ObservationRegistry;
2222

23-
import org.springframework.beans.factory.ObjectProvider;
24-
2523
/**
2624
* Registers the {@link TimerObservationHandler} with an {@link ObservationRegistry}.
2725
*
@@ -30,15 +28,15 @@
3028
class TimerObservationHandlerObservationRegistryCustomizer
3129
implements ObservationRegistryCustomizer<ObservationRegistry> {
3230

33-
private final ObjectProvider<MeterRegistry> meterRegistry;
31+
private final MeterRegistry meterRegistry;
3432

35-
TimerObservationHandlerObservationRegistryCustomizer(ObjectProvider<MeterRegistry> meterRegistry) {
33+
TimerObservationHandlerObservationRegistryCustomizer(MeterRegistry meterRegistry) {
3634
this.meterRegistry = meterRegistry;
3735
}
3836

3937
@Override
4038
public void customize(ObservationRegistry registry) {
41-
registry.observationConfig().observationHandler(new TimerObservationHandler(this.meterRegistry.getObject()));
39+
registry.observationConfig().observationHandler(new TimerObservationHandler(this.meterRegistry));
4240
}
4341

4442
}

spring-boot-project/spring-boot-actuator-autoconfigure/src/test/java/org/springframework/boot/actuate/autoconfigure/observation/TimerObservationHandlerObservationRegistryCustomizerTests.java

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,6 @@
2121
import io.micrometer.observation.Observation;
2222
import io.micrometer.observation.ObservationRegistry;
2323
import org.junit.jupiter.api.Test;
24-
import org.mockito.Mockito;
25-
26-
import org.springframework.beans.factory.ObjectProvider;
2724

2825
import static org.assertj.core.api.Assertions.assertThat;
2926

@@ -35,13 +32,10 @@
3532
class TimerObservationHandlerObservationRegistryCustomizerTests {
3633

3734
@Test
38-
@SuppressWarnings("unchecked")
3935
void customizeInstallsTimerObservationHandler() {
4036
MeterRegistry meterRegistry = new SimpleMeterRegistry();
41-
ObjectProvider<MeterRegistry> meterRegistryProvider = Mockito.mock(ObjectProvider.class);
42-
Mockito.when(meterRegistryProvider.getObject()).thenReturn(meterRegistry);
4337
TimerObservationHandlerObservationRegistryCustomizer sut = new TimerObservationHandlerObservationRegistryCustomizer(
44-
meterRegistryProvider);
38+
meterRegistry);
4539
ObservationRegistry observationRegistry = ObservationRegistry.create();
4640
sut.customize(observationRegistry);
4741
Observation.start("test-1", observationRegistry).stop();

0 commit comments

Comments
 (0)