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

Tsz Wo (Nicholas), SZE commented on HDFS-4635:
----------------------------------------------

The indentation below is off.  Otherwise, patch looks good.  +1
{code}
+ // Use 2% of total memory to size the GSet capacity
+    this.capacity = LightWeightGSet.computeCapacity(2.0, "BlocksMap");
{code}

                
> Move BlockManager#computeCapacity to LightWeightGSet
> ----------------------------------------------------
>
>                 Key: HDFS-4635
>                 URL: https://issues.apache.org/jira/browse/HDFS-4635
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: namenode
>            Reporter: Suresh Srinivas
>            Assignee: Suresh Srinivas
>             Fix For: 2.0.5-beta
>
>         Attachments: HDFS-4635.b1.patch, HDFS-4635.patch, HDFS-4635.patch, 
> HDFS-4635.patch
>
>
> The computeCapacity in BlockManager that calculates the LightWeightGSet 
> capacity as the percentage of total JVM memory should be moved to 
> LightWeightGSet. This helps in other maps that are based on the GSet to make 
> use of the same functionality.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to