You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by tr...@apache.org on 2018/12/06 11:08:27 UTC
[flink] branch release-1.6 updated: [FLINK-11011][E2E][JM] Log
error messages about null CheckpointCoordinator only if job is running
(#7223)
This is an automated email from the ASF dual-hosted git repository.
trohrmann pushed a commit to branch release-1.6
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.6 by this push:
new d9c1bf7 [FLINK-11011][E2E][JM] Log error messages about null CheckpointCoordinator only if job is running (#7223)
d9c1bf7 is described below
commit d9c1bf7a2cda85a0540f6f6c69607d0fd8c2598d
Author: azagrebin <az...@users.noreply.github.com>
AuthorDate: Thu Dec 6 12:08:20 2018 +0100
[FLINK-11011][E2E][JM] Log error messages about null CheckpointCoordinator only if job is running (#7223)
---
.../org/apache/flink/runtime/jobmaster/JobMaster.java | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobMaster.java b/flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobMaster.java
index c8a8e88..a89079d 100644
--- a/flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobMaster.java
+++ b/flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobMaster.java
@@ -681,8 +681,12 @@ public class JobMaster extends FencedRpcEndpoint<JobMasterId> implements JobMast
}
});
} else {
- log.error("Received AcknowledgeCheckpoint message for job {} with no CheckpointCoordinator",
- jobGraph.getJobID());
+ String errorMessage = "Received AcknowledgeCheckpoint message for job {} with no CheckpointCoordinator";
+ if (executionGraph.getState() == JobStatus.RUNNING) {
+ log.error(errorMessage, jobGraph.getJobID());
+ } else {
+ log.debug(errorMessage, jobGraph.getJobID());
+ }
}
}
@@ -700,8 +704,12 @@ public class JobMaster extends FencedRpcEndpoint<JobMasterId> implements JobMast
}
});
} else {
- log.error("Received DeclineCheckpoint message for job {} with no CheckpointCoordinator",
- jobGraph.getJobID());
+ String errorMessage = "Received DeclineCheckpoint message for job {} with no CheckpointCoordinator";
+ if (executionGraph.getState() == JobStatus.RUNNING) {
+ log.error(errorMessage, jobGraph.getJobID());
+ } else {
+ log.debug(errorMessage, jobGraph.getJobID());
+ }
}
}