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 >
