Plamen Jeliazkov commented on HDFS-13399:

Thanks [~xkrogen] – I've taken your changes into a new patch.

I have also added a new unit test, {{TestStateAlignmentContextWithHA}}, to 
showcase DFSClient retaining its AlignmentContext across NameNode failover 

If anyone has suggestions on how to provide additional tests or enhance 
existing ones I am open to suggestions.

I have also addressed the whitespace and checkstyle issues. There are some 
checkstyles that could not be addressed due to method overloading creating new 
methods that have more than 7 parameters.

> 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
> 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

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