[
https://issues.apache.org/jira/browse/FLINK-4364?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887712#comment-15887712
]
ASF GitHub Bot commented on FLINK-4364:
---------------------------------------
Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/3151
@tillrohrmann , I have submitted the modifications that covered all your
suggestions.
Only one issue I am not confirmed is in order to reuse the
**ScheduledExecutor** in **RpcService**, I modified the type of
**ScheduledExecutorService** in **HeartbeatManagerImpl**, otherwise we need
another wrapper like **ScheduledExecutorServiceAdapter**.
Wish your further advices and thanks in advance!
> Implement TaskManager side of heartbeat from JobManager
> -------------------------------------------------------
>
> Key: FLINK-4364
> URL: https://issues.apache.org/jira/browse/FLINK-4364
> Project: Flink
> Issue Type: Sub-task
> Components: Cluster Management
> Reporter: zhijiang
> Assignee: zhijiang
>
> The {{JobManager}} initiates heartbeat messages via (JobID, JmLeaderID), and
> the {{TaskManager}} will report metrics info for each heartbeat.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
