[ 
https://issues.apache.org/jira/browse/HDFS-14130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16728523#comment-16728523
 ] 

Konstantin Shvachko commented on HDFS-14130:
--------------------------------------------

Hi [~xiangheng]. Sounds like your patch is the right direction. This should 
prevent from converting an Observer into Standby. 
# Although I would prefer to verify {{OBSERVER}} state inside 
{{ZKFailoverController.verifyChangedServiceState()}}, which seems a more direct 
place related to this logic.
# It would be good to add a unit test with ZKFC and Observer to reproduce this 
condition.
# Do I understand correctly that with your patch Observer will still be 
considered as a target for failover? Thought may be adding Observer state check 
into {{checkEligibleForFailover()}} could solve that.

> Make ZKFC ObserverNode aware
> ----------------------------
>
>                 Key: HDFS-14130
>                 URL: https://issues.apache.org/jira/browse/HDFS-14130
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ha
>    Affects Versions: HDFS-12943
>            Reporter: Konstantin Shvachko
>            Assignee: xiangheng
>            Priority: Major
>         Attachments: HDFS-14130-HDFS-12943.001.patch
>
>
> Need to fix automatic failover with ZKFC. Currently it does not know about 
> ObserverNodes trying to convert them to SBNs.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to