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

Íñigo Goiri commented on HDFS-13981:
------------------------------------

I would be OK extracting the fields from NamenodeStatusReport into 
NamenodeSpaceReport and using it as a nested object there.
The only problem is that it looks a little overkill given that we are only 
changing the setAvailable, right?
Could we just register the namenodes and then change the available space?

BTW, for the ArrayList, do we want to create them with the proper size given we 
should know the final size?

> Review of AvailableSpaceResolver.java
> -------------------------------------
>
>                 Key: HDFS-13981
>                 URL: https://issues.apache.org/jira/browse/HDFS-13981
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: federation
>    Affects Versions: 3.2.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>            Priority: Minor
>         Attachments: HDFS-13981.1.patch, HDFS-13981.2.patch, 
> HDFS-13981.3.patch
>
>
> * No behavior changes, just optimizing and paring down the code



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to