fix: remove redundant metadata error logs within grace period (#1441)

toddbaert · web-flow · commit 7e87be34c6a3 · 2025-06-25T13:13:57.000-04:00
Signed-off-by: Todd Baert &lt;todd.baert@dynatrace.com&gt;
diff --git a/providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/connector/sync/SyncStreamQueueSource.java b/providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/connector/sync/SyncStreamQueueSource.java
@@ -141,7 +141,9 @@ private void observeSyncStream() throws InterruptedException {
 
                         metadataResponse = localStub.getMetadata(metadataRequest.build());
                     } catch (Exception metaEx) {
-                        log.error("Metadata exception: {}, cancelling stream", metaEx.getMessage(), metaEx);
+                        // cancel the stream if the getMetadata fails
+                        // we can keep this log quiet since the stream is cancelled/restarted with this exception
+                        log.debug("Metadata exception: {}, cancelling stream", metaEx.getMessage(), metaEx);
                         context.cancel(metaEx);
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,9 @@ private void observeSyncStream() throws InterruptedException {`
`141`	`141`
`142`	`142`	`metadataResponse = localStub.getMetadata(metadataRequest.build());`
`143`	`143`	`} catch (Exception metaEx) {`
`144`		`- log.error("Metadata exception: {}, cancelling stream", metaEx.getMessage(), metaEx);`
	`144`	`+ // cancel the stream if the getMetadata fails`
	`145`	`+ // we can keep this log quiet since the stream is cancelled/restarted with this exception`
	`146`	`+ log.debug("Metadata exception: {}, cancelling stream", metaEx.getMessage(), metaEx);`
`145`	`147`	`context.cancel(metaEx);`
`146`	`148`	`}`
`147`	`149`	`}`