hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (HBASE-3657) reduce copying of HRegionInfo's
Date Thu, 17 Mar 2011 20:48:29 GMT

     [ https://issues.apache.org/jira/browse/HBASE-3657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

stack resolved HBASE-3657.
--------------------------

       Resolution: Fixed
    Fix Version/s: 0.92.0
     Hadoop Flags: [Reviewed]

Applied to TRUNK.  Thank you for the patch Ted.

> reduce copying of HRegionInfo's
> -------------------------------
>
>                 Key: HBASE-3657
>                 URL: https://issues.apache.org/jira/browse/HBASE-3657
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>             Fix For: 0.92.0
>
>         Attachments: hbase-3657-with-unit-test.txt, hbase-3657.txt
>
>
> I looked at calls to this method in AssignmentManager:
>   public void assignUserRegions(List<HRegionInfo> regions, List<HServerInfo>
servers)
> Here is one from AssignmentManager:
>       assignUserRegions(Arrays.asList(regions.keySet().toArray(new HRegionInfo[0])),
servers);
> Here is another from Master:
>       this.assignmentManager.assignUserRegions(Arrays.asList(newRegions), servers);
> I propose changing the first parameter to HRegionInfo[] so that extra copying can be
avoided - considering that the number of regions in production is high.
> A bigger task is to reduce memory footprint of AssignmentManager.
> It would eliminate this copy:
>   165       public static <T> List<T> asList(T... array) {
>   166           return new ArrayList<T>(array);
>   167       }
>   168   

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message