masteryhx commented on code in PR #24474:
URL: https://github.com/apache/flink/pull/24474#discussion_r1519055013
##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/SubtaskCheckpointCoordinatorImpl.java:
##########
@@ -870,7 +870,8 @@ private static void
logCheckpointProcessingDelay(CheckpointMetaData checkpointMe
long delay = System.currentTimeMillis() -
checkpointMetaData.getReceiveTimestamp();
if (delay >= CHECKPOINT_EXECUTION_DELAY_LOG_THRESHOLD_MS) {
LOG.warn(
- "Time from receiving all checkpoint barriers/RPC to
executing it exceeded threshold: {}ms",
+ "Time from receiving all checkpoint barriers/RPC for
checkpoint id {} to executing it exceeded threshold: {}ms",
Review Comment:
```suggestion
"Time from receiving all checkpoint barriers/RPC for
checkpoint {} to executing it exceeded threshold: {}ms",
```
Maybe just use 'checkpoint' as other log info.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
