[
https://issues.apache.org/jira/browse/HBASE-12286?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14177908#comment-14177908
]
stack commented on HBASE-12286:
-------------------------------
[~manukranthk] ok on having two commands. Sounds good.
ServerName is probably more natural and aligned with how other commands run
The change to use servername maybe we should do too because looking at the
Admin api, its
void updateConfiguration(ServerName server) throws IOException;
...and this patch is doing this:
@admin.updateConfiguration(ServerName.valueOf(hostAndPort, 0));
Looking at how HBaseAdmin implements updateConfiguration, will it work when
startcode is 0?
Thanks [~manukranthk]
> [shell] Add server/cluster online load of configuration changes
> ---------------------------------------------------------------
>
> Key: HBASE-12286
> URL: https://issues.apache.org/jira/browse/HBASE-12286
> Project: HBase
> Issue Type: New Feature
> Components: shell
> Reporter: stack
> Assignee: Manukranth Kolloju
> Fix For: 0.99.2
>
> Attachments:
> 0001-Adding-shell-methods-for-Admin.updateConfiguration.patch
>
>
> Add new loadConfig and loadConfig ServerName verbs to shell to expose new
> online loading of config changes.
> This is last piece of a forward port of 0.89fb dynamic config feature done by
> [~manukranthk]
> I'll do this last bit if no one else takes it up.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
