[ 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)