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

    https://github.com/apache/flink/pull/3151#discussion_r99732047
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskExecutor.java
 ---
    @@ -1058,6 +1092,30 @@ public void handleError(Throwable throwable) {
                }
        }
     
    +   /**
    +    * The heartbeat listener for JobManager and ResourceManager, they can 
be distinguished by ResourceID
    +    * and trigger different processes.
    +    */
    +   private final class JMRMHeartbeatListener implements HeartbeatListener {
    --- End diff --
    
    From TaskExecutor side, it will monitor the JM and RM that initiate the 
heartbeat, so JMRMHeartbeatListener indicates the heartbeat timeout with JM or 
RM, the current PR just shows the interaction with JM part. I am supposed to 
submit the RM part in another PR, maybe submit both sides together will help to 
understand easily.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to