Skip to content

Commit eefbf3f

Browse files
committed
Add NOISY marker to event processing error messages
Signed-off-by: Hai Yan <[email protected]>
1 parent b033887 commit eefbf3f

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

data-prepper-plugins/rds-source/src/main/java/org/opensearch/dataprepper/plugins/source/rds/stream/BinlogEventListener.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@
5353
import java.util.function.Consumer;
5454
import java.util.stream.Collectors;
5555

56+
import static org.opensearch.dataprepper.logging.DataPrepperMarkers.NOISY;
57+
5658
public class BinlogEventListener implements BinaryLogClient.EventListener {
5759

5860
private static final Logger LOG = LoggerFactory.getLogger(BinlogEventListener.class);
@@ -464,7 +466,7 @@ private void handleEventAndErrors(com.github.shyiko.mysql.binlog.event.Event eve
464466
try {
465467
eventProcessingTimer.record(() -> function.accept(event));
466468
} catch (Exception e) {
467-
LOG.error("Failed to process change event of type {}", event.getHeader().getEventType(), e);
469+
LOG.error(NOISY, "Failed to process change event of type {}", event.getHeader().getEventType(), e);
468470
eventProcessingErrorCounter.increment();
469471
}
470472
}

data-prepper-plugins/rds-source/src/main/java/org/opensearch/dataprepper/plugins/source/rds/stream/LogicalReplicationEventProcessor.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@
4545
import java.util.Set;
4646
import java.util.function.Consumer;
4747

48+
import static org.opensearch.dataprepper.logging.DataPrepperMarkers.NOISY;
49+
4850
public class LogicalReplicationEventProcessor {
4951

5052
enum TupleDataType {
@@ -493,7 +495,7 @@ private void handleMessageWithRetries(ByteBuffer message, Consumer<ByteBuffer> f
493495
eventProcessingTimer.record(() -> function.accept(message));
494496
return;
495497
} catch (Exception e) {
496-
LOG.warn("Error when processing change event of type {}, will retry", messageType, e);
498+
LOG.warn(NOISY, "Error when processing change event of type {}, will retry", messageType, e);
497499
applyBackoff();
498500
}
499501
retry++;

0 commit comments

Comments
 (0)