AHeise commented on a change in pull request #13175:
URL: https://github.com/apache/flink/pull/13175#discussion_r473766209
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/CompletedCheckpoint.java
##########
@@ -320,6 +320,12 @@ void setDiscardCallback(@Nullable
CompletedCheckpointStats.DiscardCallback disca
@Override
public String toString() {
- return String.format("Checkpoint %d @ %d for %s", checkpointID,
timestamp, job);
+ return String.format(
+ "%s %d @ %d for %s located at %s",
+ props.getCheckpointType(),
Review comment:
Currently that would display:
"CHECKPOINT"
"SAVEPOINT"
"SYNC_SAVEPOINT"
I'm proposing to translate them into just "Checkpoint" and "Savepoint" to
reduce the technicality. However, no strong feelings, so you can also convince
me that it's better to leave as is.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
