[ 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