[
https://issues.apache.org/jira/browse/FLINK-24167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17425458#comment-17425458
]
Aitozi commented on FLINK-24167:
--------------------------------
Thanks for your suggestion, I have done the minor refactor for this. It looks
more clear than before. Please help review when you are free [~trohrmann]
> Split HeartbeatTarget to HeartbeatReceiver and HeartbeatSender
> ---------------------------------------------------------------
>
> Key: FLINK-24167
> URL: https://issues.apache.org/jira/browse/FLINK-24167
> Project: Flink
> Issue Type: Improvement
> Components: Runtime / Coordination
> Reporter: Aitozi
> Priority: Minor
> Labels: pull-request-available
>
> Currently, {{HeartbeatTarget}} is responsible for the {{receiveHeartbeat}}
> and {{requestHeartbeat}}. But many components just need one of them and let
> the not needed interface implements with {{unsupportedOperationFuture}}. This
> makes the implementation not so clear. Can we split the {{HeartbeatTarget}}
> to {{HeartbeatReceiver}} and {{HeartbeatSender}}.
> The {{HeartbeatReceiver}} only cares about the logic when receive a
> heartbeat.
> The {{HeartbeatSender}} only cares about the logic of send heartbeat.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
