[
https://issues.apache.org/jira/browse/HADOOP-7938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13188943#comment-13188943
]
Todd Lipcon commented on HADOOP-7938:
-------------------------------------
{code}
+ // Try to fence fromSvc
+ if (fencer != null) {
+ if (!fencer.fence()) {
+ throw new FailoverFailedException("Unable to fence " + fromSvcName);
+ }
}
{code}
Shouldn't you only fence the old node in the case that you got an exception in
the {{transitionToStandby}} call? Then, you also wouldn't need to make the user
specify whether or not to fence - it would automatically fence in the case that
there was a problem with graceful failover.
----
- If the {{transitionToActive}} call fails, we need to be careful before doing
a failback. For example, what if it was a timeout, and in fact the new active
is still in the process of failing over? Then we need to fence "toSvc" before
going back to "fromSvc"
> HA: the FailoverController should optionally fence the active during failover
> -----------------------------------------------------------------------------
>
> Key: HADOOP-7938
> URL: https://issues.apache.org/jira/browse/HADOOP-7938
> Project: Hadoop Common
> Issue Type: Sub-task
> Components: ha
> Affects Versions: HA Branch (HDFS-1623)
> Reporter: Eli Collins
> Assignee: Eli Collins
> Fix For: HA Branch (HDFS-1623)
>
> Attachments: hadoop-7938.txt
>
>
> The FailoverController in HADOOP-7924 needs to be able to fence off the
> current active in case it fails to transition to standby (or the user
> requests it for sanity). This is needed even for manual failover (the CLI
> should use the configured fencing mechanism). The FC needs to access the
> HDFS-specific implementations HDFS-2179, could add a common fencing interface
> (or just shell out but we may not always want to do that).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira