Set refreshError to false if ExternalMapping has not changed (#302)

matvey-mtn · web-flow · commit bc804b51afbc · 2022-04-03T09:45:44.000+03:00
diff --git a/sawmill-core/src/main/java/io/logz/sawmill/processors/ExternalMappingSourceProcessor.java b/sawmill-core/src/main/java/io/logz/sawmill/processors/ExternalMappingSourceProcessor.java
@@ -72,6 +72,7 @@ protected void refreshExternalMapping() {
             ExternalMappingResponse externalMappingResponse = externalMappingsClient.loadMappings(lastModifiedTime.get());
             if (!externalMappingResponse.isModified()) {
                 logger.debug("External mapping didn't change since {}. Skipping refresh.", lastModifiedTime.get());
+                refreshErrorOccurred = false;
                 return;
             }
 

Original file line number	Diff line number	Diff line change
`@@ -72,6 +72,7 @@ protected void refreshExternalMapping() {`
`72`	`72`	`ExternalMappingResponse externalMappingResponse = externalMappingsClient.loadMappings(lastModifiedTime.get());`
`73`	`73`	`if (!externalMappingResponse.isModified()) {`
`74`	`74`	`logger.debug("External mapping didn't change since {}. Skipping refresh.", lastModifiedTime.get());`
	`75`	`+ refreshErrorOccurred = false;`
`75`	`76`	`return;`
`76`	`77`	`}`
`77`	`78`