andreacavalli/scylla-jmx
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

StorageService: remove node should use hostid and not token

Browse Source 
There is a confusion in origin, the MBean declare token as the parameter
to remove, but the implementation actually uses host id.

This patch modify scylla implementation to pass a host id as the
parameter to remove.

Signed-off-by: Amnon Heiman <amnon@cloudius-systems.com>
This commit is contained in:
Amnon Heiman 2015-10-26 09:41:31 +02:00 committed by Pekka Enberg
parent dc76d2550e
commit a707b487f0
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/main/java/org/apache/cassandra/service/StorageService.java
View File

@ -689,11 +689,12 @@ public class StorageService extends NotificationBroadcasterSupport
/**
* removeToken removes token (and all data associated with enpoint that had
* it) from the ring
* @param hostIdString the host id to remove
*/
public void removeNode(String token) {
public void removeNode(String hostIdString) {
log(" removeNode(String token)");
MultivaluedMap<String, String> queryParams = new MultivaluedMapImpl();
APIClient.set_query_param(queryParams, "new_token", token);
APIClient.set_query_param(queryParams, "host_id", hostIdString);
c.post("/storage_service/remove_node", queryParams);
}