Skip to content

Commit 0a81a14

Browse files
authoredMar 10, 2025··
chore: adjust noisy log lines (#1268)
Signed-off-by: Todd Baert <[email protected]>

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed
 

‎providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/FlagdProvider.java

+7-6
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ EvaluationContext getEnrichedContext() {
193193

194194
@SuppressWarnings("checkstyle:fallthrough")
195195
private void onProviderEvent(FlagdProviderEvent flagdProviderEvent) {
196-
log.info("FlagdProviderEvent event {} ", flagdProviderEvent.getEvent());
196+
log.debug("FlagdProviderEvent event {} ", flagdProviderEvent.getEvent());
197197
synchronized (syncResources) {
198198
/*
199199
* We only use Error and Ready as previous states.
@@ -231,7 +231,7 @@ private void onProviderEvent(FlagdProviderEvent flagdProviderEvent) {
231231
break;
232232

233233
default:
234-
log.info("Unknown event {}", flagdProviderEvent.getEvent());
234+
log.warn("Unknown event {}", flagdProviderEvent.getEvent());
235235
}
236236
}
237237
}
@@ -245,7 +245,7 @@ private void onConfigurationChanged(FlagdProviderEvent flagdProviderEvent) {
245245

246246
private void onReady() {
247247
if (syncResources.initialize()) {
248-
log.info("initialized FlagdProvider");
248+
log.info("Initialized FlagdProvider");
249249
}
250250
if (errorTask != null && !errorTask.isCancelled()) {
251251
errorTask.cancel(false);
@@ -256,8 +256,9 @@ private void onReady() {
256256
}
257257

258258
private void onError() {
259-
log.info("Connection lost. Emit STALE event...");
260-
log.debug("Waiting {}s for connection to become available...", gracePeriod);
259+
log.debug(
260+
"Stream error. Emitting STALE, scheduling ERROR, and waiting {}s for connection to become available.",
261+
gracePeriod);
261262
this.emitProviderStale(ProviderEventDetails.builder()
262263
.message("there has been an error")
263264
.build());
@@ -271,7 +272,7 @@ private void onError() {
271272
() -> {
272273
if (syncResources.getPreviousEvent() == ProviderEvent.PROVIDER_ERROR) {
273274
log.debug(
274-
"Provider did not reconnect successfully within {}s. Emit ERROR event...",
275+
"Provider did not reconnect successfully within {}s. Emitting ERROR event...",
275276
gracePeriod);
276277
flagResolver.onError();
277278
this.emitProviderError(ProviderEventDetails.builder()

‎providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/resolver/process/InProcessResolver.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -68,18 +68,18 @@ public void init() throws Exception {
6868
flagStore.getStateQueue().take();
6969
switch (storageStateChange.getStorageState()) {
7070
case OK:
71-
log.info("onConnectionEvent.accept ProviderEvent.PROVIDER_CONFIGURATION_CHANGED");
71+
log.debug("onConnectionEvent.accept ProviderEvent.PROVIDER_CONFIGURATION_CHANGED");
7272
onConnectionEvent.accept(new FlagdProviderEvent(
7373
ProviderEvent.PROVIDER_CONFIGURATION_CHANGED,
7474
storageStateChange.getChangedFlagsKeys(),
7575
storageStateChange.getSyncMetadata()));
76-
log.info("post onConnectionEvent.accept ProviderEvent.PROVIDER_CONFIGURATION_CHANGED");
76+
log.debug("post onConnectionEvent.accept ProviderEvent.PROVIDER_CONFIGURATION_CHANGED");
7777
break;
7878
case ERROR:
7979
onConnectionEvent.accept(new FlagdProviderEvent(ProviderEvent.PROVIDER_ERROR));
8080
break;
8181
default:
82-
log.info(String.format(
82+
log.warn(String.format(
8383
"Storage emitted unhandled status: %s", storageStateChange.getStorageState()));
8484
}
8585
}

‎providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/FlagStore.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ private void streamerListener(final QueueSource connector) throws InterruptedExc
143143
}
144144
break;
145145
default:
146-
log.info(String.format("Payload with unknown type: %s", payload.getType()));
146+
log.warn(String.format("Payload with unknown type: %s", payload.getType()));
147147
}
148148
}
149149

0 commit comments

Comments
 (0)
Please sign in to comment.