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:46:57 GMT
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 <lars.george@gmail.com> wrote:
> 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