I also added this for convenience: https://issues.apache.org/jira/browse/HBASE-17391
On Thu, Dec 29, 2016 at 4:32 PM, Lars George <[email protected]> wrote: > Done: https://issues.apache.org/jira/browse/HBASE-17390 > > > > On Thu, Dec 29, 2016 at 4:05 PM, Ted Yu <[email protected]> wrote: >> Good finding. >> >> getServers() only returns the live servers. getMaster() and >> getBackupMasters() >> should also be called. >> >> Lars: >> Mind creating a JIRA ? >> >> On Thu, Dec 29, 2016 at 6:46 AM, Lars George <[email protected]> wrote: >> >>> Hi, >>> >>> It seems that the update command is only iterating over the region servers: >>> >>> public void updateConfiguration() throws IOException { >>> for (ServerName server : this.getClusterStatus().getServers()) { >>> updateConfiguration(server); >>> } >>> } >>> >>> This leaves out the masters, which you have to trigger manually. In >>> addition the real master name is hard to find, as the UI is stripping >>> of the port and timestamp part. I had to use zk_dump to get the >>> current master server name to send an explicit "update_config" to it. >>> >>> Should the "update_all_config" not include the masters? I would say so. >>> >>> Thoughts? >>> >>> Cheers, >>> Lars >>>
