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

Mikhail Antonov commented on HBASE-11219:
-----------------------------------------

*patch looks good to me

> HRegionServer#createRegionLoad() should reuse RegionLoad.Builder instance 
> when called in a loop
> -----------------------------------------------------------------------------------------------
>
>                 Key: HBASE-11219
>                 URL: https://issues.apache.org/jira/browse/HBASE-11219
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 11219-v1.txt
>
>
> As Andrew showed in the attachment to HBASE-11165, 
> ClusterStatusProtos$RegionLoad$Builder took 25MB heap space.
> buildServerLoad() calls createRegionLoad() for each region on the region 
> server.
> One instance of ClusterStatusProtos$RegionLoad$Builder can be created outside 
> the loop and passed to createRegionLoad().
> Thanks Andrew for the finding.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to