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

Adam J Shook edited comment on ACCUMULO-4535 at 12/15/16 8:08 PM:
------------------------------------------------------------------

Wouldn't it be best to remove {{TabletBalancer#init(ServerConfiguration)}} 
entirely?  I find no other usages of this function outside of the 
{{HostRegexTableLoadBalancer}} tests and in the {{ChaoticLoadBalancer}} which 
just throws a not implemented exception.

edit: edit: Nevermind the nevermind, this could be done...  Though it would 
break custom load balancers that are using this function instead of the other 
one.


was (Author: adamjshook):
Wouldn't it be best to remove {{TabletBalancer#init(ServerConfiguration)}} 
entirely?  I find no other usages of this function outside of the 
{{HostRegexTableLoadBalancer}} tests and in the {{ChaoticLoadBalancer}} which 
just throws a not implemented exception.

edit: edit: Nevermind the nevermind, this could be done...  Though it would 
break non-standard load balancers that are using this function instead of the 
other one.

> HostRegexTableLoadBalancer fails with NullPointerException
> ----------------------------------------------------------
>
>                 Key: ACCUMULO-4535
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4535
>             Project: Accumulo
>          Issue Type: Bug
>    Affects Versions: 1.7.2
>            Reporter: Adam J Shook
>            Assignee: Adam J Shook
>
> Ran into an issue when testing out the {{HostRegexTableLoadBalancer}}.  It 
> fails to start and throws a {{NullPointerException}} when trying to access 
> the {{poolNameToRegexPattern}}.  The root cause is the {{init}} function the 
> load balancer overrides is the wrong one that the Master calls (wrong 
> signature).



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

Reply via email to