From 8d7cd07d6c90cbe608ae07abfe6528989af340c0 Mon Sep 17 00:00:00 2001 From: Asias He Date: Wed, 13 Apr 2016 08:47:59 +0800 Subject: [PATCH] storage_service: Add print info in confirm_replication The message is rare but it is very useful to debug removenode operation. --- service/storage_service.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/service/storage_service.cc b/service/storage_service.cc index 4b7b5bf7a8..4e83626305 100644 --- a/service/storage_service.cc +++ b/service/storage_service.cc @@ -2358,6 +2358,8 @@ future<> storage_service::send_replication_notification(inet_address remote) { future<> storage_service::confirm_replication(inet_address node) { return run_with_no_api_lock([node] (storage_service& ss) { + auto removing_node = bool(ss._removing_node) ? sprint("%s", *ss._removing_node) : "NONE"; + logger.info("Got confirm_replication from {}, removing_node {}", node, removing_node); // replicatingNodes can be empty in the case where this node used to be a removal coordinator, // but restarted before all 'replication finished' messages arrived. In that case, we'll // still go ahead and acknowledge it.