From ef46a7f80280ced12db35af0faccc32fb96d6fab Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 5 Aug 2025 15:08:37 +0000 Subject: [PATCH 1/2] chore: bump com.github.spotbugs:spotbugs-maven-plugin Bumps [com.github.spotbugs:spotbugs-maven-plugin](https://github.com/spotbugs/spotbugs-maven-plugin) from 4.8.4.0 to 4.9.3.2. - [Release notes](https://github.com/spotbugs/spotbugs-maven-plugin/releases) - [Commits](https://github.com/spotbugs/spotbugs-maven-plugin/compare/spotbugs-maven-plugin-4.8.4.0...spotbugs-maven-plugin-4.9.3.2) --- updated-dependencies: - dependency-name: com.github.spotbugs:spotbugs-maven-plugin dependency-version: 4.9.3.2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index a1b714540..9fa64a616 100644 --- a/pom.xml +++ b/pom.xml @@ -586,7 +586,7 @@ com.github.spotbugs spotbugs-maven-plugin - 4.8.4.0 + 4.9.3.2 test From 59528e41e79ea4f256aab65aaf416e17a0e88d0b Mon Sep 17 00:00:00 2001 From: Philipp Page Date: Fri, 8 Aug 2025 11:07:54 +0200 Subject: [PATCH 2/2] Make raiseOnEmptyMetrics an AtomicBoolean. --- .../powertools/metrics/internal/EmfMetricsLogger.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/internal/EmfMetricsLogger.java b/powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/internal/EmfMetricsLogger.java index 1eedd270d..2f6f9e689 100644 --- a/powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/internal/EmfMetricsLogger.java +++ b/powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/internal/EmfMetricsLogger.java @@ -50,7 +50,7 @@ public class EmfMetricsLogger implements Metrics { private final software.amazon.cloudwatchlogs.emf.logger.MetricsLogger emfLogger; private final EnvironmentProvider environmentProvider; - private boolean raiseOnEmptyMetrics = false; + private AtomicBoolean raiseOnEmptyMetrics = new AtomicBoolean(false); private String namespace; private Map defaultDimensions = new HashMap<>(); private final AtomicBoolean hasMetrics = new AtomicBoolean(false); @@ -133,7 +133,7 @@ public void setNamespace(String namespace) { @Override public void setRaiseOnEmptyMetrics(boolean raiseOnEmptyMetrics) { - this.raiseOnEmptyMetrics = raiseOnEmptyMetrics; + this.raiseOnEmptyMetrics.set(raiseOnEmptyMetrics); } @Override @@ -159,7 +159,7 @@ public void flush() { Validator.validateNamespace(namespace); if (!hasMetrics.get()) { - if (raiseOnEmptyMetrics) { + if (raiseOnEmptyMetrics.get()) { throw new IllegalStateException("No metrics were emitted"); } else { LOGGER.warn("No metrics were emitted");