[
https://issues.apache.org/jira/browse/FLINK-7655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16175609#comment-16175609
]
ASF GitHub Bot commented on FLINK-7655:
---------------------------------------
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4689
Travis passed. Merging this PR.
> Revisit default non-leader id for FencedRpcEndpoints
> ----------------------------------------------------
>
> Key: FLINK-7655
> URL: https://issues.apache.org/jira/browse/FLINK-7655
> Project: Flink
> Issue Type: Bug
> Components: Distributed Coordination
> Affects Versions: 1.4.0
> Reporter: Till Rohrmann
> Assignee: Till Rohrmann
> Priority: Minor
> Labels: flip-6
>
> Currently, when a {{FencedRpcEndpoint}} loses leadership, we set its leader
> id to a random value. This can be problematic, even though it's unlikely,
> because we might set it to a value which is used somewhere else (e.g. the
> currently valid leader id). I think it would be better to simply set the
> leader id to {{null}} in order to properly encode that the
> {{FencedRpcEndpoint}} is no longer a leader.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
