You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by se...@apache.org on 2015/02/09 15:57:12 UTC
flink git commit: [FLINK-1478] [jobmanager] (addendum) Add eager
check for null as input split host name
Repository: flink
Updated Branches:
refs/heads/master 4386620c0 -> c1e1aca21
[FLINK-1478] [jobmanager] (addendum) Add eager check for null as input split host name
Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/c1e1aca2
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/c1e1aca2
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/c1e1aca2
Branch: refs/heads/master
Commit: c1e1aca218b0a35e105b8430acaa36bbe7f04b4a
Parents: 4386620
Author: Stephan Ewen <se...@apache.org>
Authored: Mon Feb 9 15:56:13 2015 +0100
Committer: Stephan Ewen <se...@apache.org>
Committed: Mon Feb 9 15:56:13 2015 +0100
----------------------------------------------------------------------
.../apache/flink/runtime/executiongraph/ExecutionJobVertex.java | 4 ++++
1 file changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/flink/blob/c1e1aca2/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionJobVertex.java
----------------------------------------------------------------------
diff --git a/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionJobVertex.java b/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionJobVertex.java
index 45e3245..0a5a463 100644
--- a/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionJobVertex.java
+++ b/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionJobVertex.java
@@ -481,6 +481,10 @@ public class ExecutionJobVertex implements Serializable {
throw new JobException("Strictly local assignment requires exactly one hostname for each LocatableInputSplit.");
}
String hostName = lis.getHostnames()[0];
+
+ if (hostName == null) {
+ throw new JobException("For strictly local input split assignment, no null host names are allowed.");
+ }
List<LocatableInputSplit> hostSplits = splitsByHost.get(hostName);
if (hostSplits == null) {