[
https://issues.apache.org/jira/browse/HADOOP-7380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13081343#comment-13081343
]
Aaron T. Myers commented on HADOOP-7380:
----------------------------------------
bq. Aaron, should this change have been made in HA branch.
Eli seemed to think this change was sufficiently generic that it could go into
trunk before the HA branch was cut. I'm inclined to agree with him, but if you
feel strongly, we could certainly back it out of trunk and commit it only to
the HA branch. I don't see much benefit in doing so versus just addressing any
concerns you have in subsequent patches, but I don't feel strongly about that.
bq. Also it would be good to add a little design document.
Sure, I can work on that.
bq. This is a fundamental change in the IPC layer. I have concerns on adding
failover, retry etc. all in one patch.
I think you're overstating the extent of this change a little bit. This patch
doesn't introduce retry - all of that was already in place. This patch augments
the existing retry facility to optionally fail over and retry on a different
proxy object.
bq. It would help reviews if this is done in a smaller patches.
Certainly, I can do that in the future.
> Add client failover functionality to o.a.h.io.(ipc|retry)
> ---------------------------------------------------------
>
> Key: HADOOP-7380
> URL: https://issues.apache.org/jira/browse/HADOOP-7380
> Project: Hadoop Common
> Issue Type: Sub-task
> Components: ipc
> Affects Versions: 0.23.0
> Reporter: Aaron T. Myers
> Assignee: Aaron T. Myers
> Fix For: 0.23.0
>
> Attachments: hadoop-7380-hdfs-example.patch, hadoop-7380.0.patch,
> hadoop-7380.1.patch, hadoop-7380.2.patch, hdfs-7380.3.patch
>
>
> Implementing client failover will likely require changes to {{o.a.h.io.ipc}}
> and/or {{o.a.h.io.retry}}. This JIRA is to track those changes.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira