hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ted Yu <yuzhih...@gmail.com>
Subject Re: update_all_config and masters
Date Thu, 29 Dec 2016 15:05:27 GMT
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 <lars.george@gmail.com> 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
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message