[jira] [Commented] (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:comment-tabpanel=16416639#comment-16416639 ] Hudson commented on HBASE-20133: Results for branch HBASE-19064 [build #77 on

[jira] [Commented] (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:comment-tabpanel=16396876#comment-16396876 ] Xiang Li commented on HBASE-20133: -- [~yuzhih...@gmail.com], thanks for helping to commit the patch!

[jira] [Commented] (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:comment-tabpanel=16394493#comment-16394493 ] Hudson commented on HBASE-20133: Results for branch master [build #259 on

[jira] [Commented] (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:comment-tabpanel=16393850#comment-16393850 ] Ted Yu commented on HBASE-20133: Adding debug log next to the line above, I found coverage by

[jira] [Commented] (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:comment-tabpanel=16393186#comment-16393186 ] Hadoop QA commented on HBASE-20133: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (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:comment-tabpanel=16393168#comment-16393168 ] Ted Yu commented on HBASE-20133: With latest patch, I placed a breakpoint on the last line below: {code}

[jira] [Commented] (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:comment-tabpanel=16393160#comment-16393160 ] Xiang Li commented on HBASE-20133: -- By the way, in

[jira] [Commented] (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:comment-tabpanel=16393133#comment-16393133 ] Xiang Li commented on HBASE-20133: -- Hi [~yuzhih...@gmail.com], I uploaded patch 002 to address the

[jira] [Commented] (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:comment-tabpanel=16392361#comment-16392361 ] Ted Yu commented on HBASE-20133: Can you address checkstyle warnings ? Is the changed method covered by

[jira] [Commented] (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:comment-tabpanel=16392282#comment-16392282 ] Hadoop QA commented on HBASE-20133: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (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:comment-tabpanel=16391242#comment-16391242 ] Xiang Li commented on HBASE-20133: -- [~yuzhih...@gmail.com] I upload patch 001, in which

[jira] [Commented] (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:comment-tabpanel=16389711#comment-16389711 ] Ted Yu commented on HBASE-20133: We have org.apache.hadoop.hbase.util.Pair Thanks > Calculate correct

[jira] [Commented] (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:comment-tabpanel=16389682#comment-16389682 ] Xiang Li commented on HBASE-20133: -- Thanks [~yuzhih...@gmail.com] for the comments! 1. Regarding bq. BTW

[jira] [Commented] (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:comment-tabpanel=16388078#comment-16388078 ] Ted Yu commented on HBASE-20133: bq. RegionPlan is returned by the input parameter Have you considered

[jira] [Commented] (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:comment-tabpanel=16387752#comment-16387752 ] Hadoop QA commented on HBASE-20133: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (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:comment-tabpanel=16387596#comment-16387596 ] Xiang Li commented on HBASE-20133: -- Uploaded patch 000 for review. # correctAssignments() is updated to