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

Zhihong Ted Yu commented on HBASE-5953:
---------------------------------------

lgtm.
Minor nit:
{code}
+      this.getCoprocessors(),this.balanceSwitch);
{code}
Insert space after comma - will be done at integration time.
                
> Expose the current state of the balancerSwitch
> ----------------------------------------------
>
>                 Key: HBASE-5953
>                 URL: https://issues.apache.org/jira/browse/HBASE-5953
>             Project: HBase
>          Issue Type: Improvement
>          Components: master
>    Affects Versions: 0.96.0
>            Reporter: David S. Wang
>            Assignee: Gregory Chanan
>            Priority: Minor
>         Attachments: HBASE-5953.patch
>
>
> The balancerSwitch as in both 0.90 and 0.92.1 is implemented in such a way 
> that it is impossible to retrieve its value without changing it:
> /**
> Turn the load balancer on or off.
> @param b If true, enable balancer. If false, disable balancer.
> @return Previous balancer value
> */
> public boolean balanceSwitch(final boolean b);
> It would be nice to have a way to just get the current state.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to