[
https://issues.apache.org/jira/browse/HDFS-13399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16448619#comment-16448619
]
Plamen Jeliazkov commented on HDFS-13399:
-----------------------------------------
Thanks for the review, [~shv].
Regarding (1), those changes are pretty easy and I generally agree. I think
this was the sort of approach I was looking towards when I asked about
configuration; minimizing change of behaviors to only when necessary.
Regarding (2), assuming we implement (1), then in the case of most
instantiations of AbstractNNFailoverProxyProvider we will simply pass null for
an {{AlignmentContext}}. Only if DFSClient's initialization will end up calling
{{NameNodeProxiesClient.createHAProxy}} will we set the {{AlignmentContext}}.
Is this acceptable? It will be difficult otherwise to pass an
{{AlignmentContext}} into {{Client}} itself.
We can change NameNodeProxiesClient.createProxyWithClientProtocol like so:
{code:java}
if (failoverProxyProvider == null) {
...normal case...
} else {
failoverProxyProvider.setAlignmentContext(alignmentContext);
return createHAProxy(conf, nameNodeUri, ClientProtocol.class,
failoverProxyProvider);
}{code}
> Make Client field AlignmentContext non-static.
> ----------------------------------------------
>
> Key: HDFS-13399
> URL: https://issues.apache.org/jira/browse/HDFS-13399
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Affects Versions: HDFS-12943
> Reporter: Plamen Jeliazkov
> Assignee: Plamen Jeliazkov
> Priority: Major
> Attachments: HDFS-13399-HDFS-12943.000.patch,
> HDFS-13399-HDFS-12943.001.patch, HDFS-13399-HDFS-12943.002.patch
>
>
> In HDFS-12977, DFSClient's constructor was altered to make use of a new
> static method in Client that allowed one to set an AlignmentContext. This
> work is to remove that static field and make each DFSClient pass it's
> AlignmentContext down to the proxy Call level.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
