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

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

I'd like to clarify a question from [~arp] regarding the modifications to 
ECBlockGroupStats.
This JIRA brought up an issue with reporting back ECBlockGroupStats.
The main issue was that when merging ECBlockGroupStats, we use a HashMap with 
the stats from each subcluster.
With the old implementation it was inserting the same stats multiple times and 
accounting it wrongly.
This was the reason to implement {{hashCode()}} and {{equals()}}.

The other change is the addition of a static merge method.
This seemed like a natural place for it to be in case others wanted to do the 
same operation outside of RBF.
Obviously, this could go away and leave it where it was but seemed like a 
reasonable addition.

> RBF: Fix the location of the DNs in getDatanodeReport()
> -------------------------------------------------------
>
>                 Key: HDFS-14268
>                 URL: https://issues.apache.org/jira/browse/HDFS-14268
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Íñigo Goiri
>            Assignee: Íñigo Goiri
>            Priority: Major
>             Fix For: HDFS-13891
>
>         Attachments: HDFS-14268-HDFS-13891.000.patch, 
> HDFS-14268-HDFS-13891.001.patch, HDFS-14268-HDFS-13891.002.patch, 
> HDFS-14268-HDFS-13891.003.patch, HDFS-14268-HDFS-13891.004.patch
>
>
> When getting all the DNs in the federation, the Router queries each of the 
> subclusters and aggregates them assigning the subcluster id to the location. 
> This query uses a {{HashSet}} which provides a "random" order for the results.



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