Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5548#discussion_r169943180
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/Execution.java
---
@@ -377,14 +377,14 @@ public boolean scheduleForExecution() {
* @param queued Flag to indicate whether the scheduler may queue this
task if it cannot
* immediately deploy it.
* @param locationPreferenceConstraint constraint for the location
preferences
- * @throws IllegalStateException Thrown, if the vertex is not in
CREATED state, which is the only state that permits scheduling.
+ * @returns Future which is completed once the Execution has been
deployed
--- End diff --
Should be `@return` not `@returns`.
---
