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

stack commented on HBASE-17194:
-------------------------------

There seems to be no relation between the current getOnlineServersList and the 
new one below....

1096      public List<ServerName> getOnlineServersList(List<ServerName> keys,
1097        BiFunction<ServerName, ServerLoad, Boolean> predicater) {


Should the new one call the old one internally at least? Or iterate 
this.onlineServers?

Does this happen alot?

Otherwise the patch looks good to me.





> Assign the new region to the idle server after splitting
> --------------------------------------------------------
>
>                 Key: HBASE-17194
>                 URL: https://issues.apache.org/jira/browse/HBASE-17194
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 2.0.0
>            Reporter: ChiaPing Tsai
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: HBASE-17194.v0.patch, evaluation-v0.png
>
>
> The new regions are assigned to the random servers after splitting, but there 
> always are some idle servers which don’t be assigned any regions on the new 
> cluster. It is a bad start of load balance, hence we should give priority to 
> the idle servers for assignment.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to