fix: fix skip logging on exporter closing

mutianf · mutianf · commit 2b829c6e47d0 · 2026-04-16T09:25:50.000-04:00
Change-Id: I6fbe6935d51dfb67aaba5500ca391abc5fec9401
diff --git a/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/exporter/BigtableCloudMonitoringExporter.java b/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/exporter/BigtableCloudMonitoringExporter.java
@@ -195,7 +195,7 @@ public void onFailure(Throwable throwable) {
               RuntimeException asyncWrapper = new RuntimeException("export failed", throwable);
               asyncWrapper.setStackTrace(stackTrace);
 
-              if (state.get() != State.Closing || state.get() != State.Closed) {
+              if (state.get() != State.Closing && state.get() != State.Closed) {
                 // ignore the export warning when client is shutting down
                 LOGGER.log(Level.WARNING, msg, asyncWrapper);
               }

Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ public void onFailure(Throwable throwable) {`
`195`	`195`	`RuntimeException asyncWrapper = new RuntimeException("export failed", throwable);`
`196`	`196`	`asyncWrapper.setStackTrace(stackTrace);`
`197`	`197`
`198`		`- if (state.get() != State.Closing \|\| state.get() != State.Closed) {`
	`198`	`+ if (state.get() != State.Closing && state.get() != State.Closed) {`
`199`	`199`	`// ignore the export warning when client is shutting down`
`200`	`200`	`LOGGER.log(Level.WARNING, msg, asyncWrapper);`
`201`	`201`	`}`