Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
MIME-Version: 1.0
From: Lars George <lars.george@gmail.com>
Date: Thu, 29 Dec 2016 15:46:55 +0100
Message-ID: <CAJL+TURBoPzAHjsdojyp3Wr63GCPYDQa+6MXmkbRvmKdkPVSPA@mail.gmail.com>
Subject: update_all_config and masters
To: "dev@hbase.apache.org" <dev@hbase.apache.org>
Content-Type: text/plain; charset=UTF-8
archived-at: Thu, 29 Dec 2016 14:47:15 -0000

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