[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ted Yu updated HBASE-20133: --- Resolution: Fixed Status: Resolved (was: Patch Available) Thanks for the patch, Xiang. > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Fix For: 3.0.0 > > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch, HBASE-20133.master.002.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ted Yu updated HBASE-20133: --- Hadoop Flags: Reviewed Fix Version/s: 3.0.0 > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Fix For: 3.0.0 > > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch, HBASE-20133.master.002.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Attachment: HBASE-20133.master.002.patch > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch, HBASE-20133.master.002.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Status: Patch Available (was: Open) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch, HBASE-20133.master.002.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Status: Open (was: Patch Available) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch, HBASE-20133.master.002.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Status: Patch Available (was: Open) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Attachment: HBASE-20133.master.001.patch > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch, > HBASE-20133.master.001.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Status: Open (was: Patch Available) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Priority: Minor (was: Critical) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Minor > Attachments: HBASE-20133.master.000.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Status: Patch Available (was: Open) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Critical > Attachments: HBASE-20133.master.000.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Attachment: HBASE-20133.master.000.patch > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Critical > Attachments: HBASE-20133.master.000.patch > > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-20133) Calculate correct assignment and build region movement plans for mis-placed regions in one pass
[ https://issues.apache.org/jira/browse/HBASE-20133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiang Li updated HBASE-20133: - Summary: Calculate correct assignment and build region movement plans for mis-placed regions in one pass (was: Calculate corrected assignment and build region movement plans for mis-placed regions in one pass) > Calculate correct assignment and build region movement plans for mis-placed > regions in one pass > --- > > Key: HBASE-20133 > URL: https://issues.apache.org/jira/browse/HBASE-20133 > Project: HBase > Issue Type: Improvement > Components: rsgroup >Reporter: Xiang Li >Assignee: Xiang Li >Priority: Critical > > In RSGroupBasedLoadBalancer#balanceCluster(clusterState), the logic could be > improved: > correctAssignment() builds a map for mis-placed and placed regions. For > mis-placed regions, the key(ServerName) is BOGUS_SERVER_NAME. Then the logic > gets those mis-paced regions out and calls findServerForRegion() several > times to find out the current host server, in order to build RegionPlan for > movement. > Some logic in correctAssignment() and findServerForRegion() could be merged > so as to build both corrected assignment and RegionPlan for mis-placed region > in one pass. As a result, findServerForRegion() could be removed if I get it > correctly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)