hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Lars George <lars.geo...@gmail.com>
Subject Re: update_all_config and masters
Date Thu, 29 Dec 2016 15:32:24 GMT
Done: https://issues.apache.org/jira/browse/HBASE-17390



On Thu, Dec 29, 2016 at 4:05 PM, Ted Yu <yuzhihong@gmail.com> 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 <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
View raw message