[GitHub] flink pull request #6103: [FLINK-9413] [distributed coordination] Tasks can ...

[zhangminglei]

Github user zhangminglei commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6103#discussion_r192559985
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/deployment/InputChannelDeploymentDescriptor.java
 ---
    @@ -103,9 +103,7 @@ public String toString() {
                        // The producing task needs to be RUNNING or already 
FINISHED
                        if (consumedPartition.isConsumable() && producerSlot != 
null &&
                                        (producerState == 
ExecutionState.RUNNING ||
    -                                           producerState == 
ExecutionState.FINISHED ||
    -                                           producerState == 
ExecutionState.SCHEDULED ||
    -                                           producerState == 
ExecutionState.DEPLOYING)) {
    --- End diff --
    
    Yes Till. The flink1.5 code does not use JobManager and TaskManager class 
for running. Instead, we use JobMaster & TaskExecutor now. I think we should 
remove them and the relative class for legacy code.


---