fix: use ConcurrentHashMap instead of HashMap (#856)

Close: #847
micronaut-projects · Oct 29, 2024 · bb95485 · bb95485
1 parent 3541062
commit bb95485
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...meter-core/src/main/java/io/micronaut/configuration/metrics/annotation/MetricOptions.java b/...meter-core/src/main/java/io/micronaut/configuration/metrics/annotation/MetricOptions.java
@@ -26,7 +26,7 @@
 import static java.lang.annotation.RetentionPolicy.RUNTIME;
 
 /**
- * Holds metadata about metric options to apply
+ * Holds metadata about metric options to apply.
  *
  * @author Haiden Rothwell
  * @since 5.6.0

diff --git a/...ore/src/main/java/io/micronaut/configuration/metrics/binder/retry/RetryMetricsBinder.java b/...ore/src/main/java/io/micronaut/configuration/metrics/binder/retry/RetryMetricsBinder.java
@@ -27,8 +27,8 @@
 import jakarta.inject.Singleton;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-
-import java.util.HashMap;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
 import static io.micronaut.configuration.metrics.micrometer.MeterRegistryFactory.MICRONAUT_METRICS_BINDERS;
 
@@ -46,7 +46,7 @@ public class RetryMetricsBinder implements RetryEventListener {
     public static final String RETRY_METRICS_ENABLED = MICRONAUT_METRICS_BINDERS + ".retry.enabled";
     private static final Logger LOGGER = LoggerFactory.getLogger(RetryMetricsBinder.class);
     private final BeanProvider<MeterRegistry> meterRegistryProvider;
-    private final HashMap<ExecutableMethod<?, ?>, Counter> attemptCounters = new HashMap<>();
+    private final Map<ExecutableMethod<?, ?>, Counter> attemptCounters = new ConcurrentHashMap<>();
 
     /**
      * @param meterRegistryProvider The meter registry provider