From 3f0c70f2d56743707699f6a5b62a1c2c2d5c6f85 Mon Sep 17 00:00:00 2001 From: Sakshi Bansal Date: Thu, 12 Nov 2015 12:28:53 +0530 Subject: afr: replica pair going offline does not require CHILD_MODIFIED event As a part of CHILD_MODIFIED event DHT forgets the current layout and performs fresh lookup. However this is not required when a replica pair goes offline as the xattrs can be read from other replica pairs. Hence setting different event to handle replica pair going down. Change-Id: I5ede2a6398e63f34f89f9d3c9bc30598974402e3 BUG: 1281230 Signed-off-by: Sakshi Bansal Reviewed-on: http://review.gluster.org/12573 Reviewed-by: Ravishankar N Reviewed-by: Susant Palai Tested-by: NetBSD Build System Tested-by: Gluster Build System Reviewed-by: Jeff Darcy --- xlators/cluster/afr/src/afr-common.c | 2 +- xlators/cluster/dht/src/dht-common.c | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) (limited to 'xlators') diff --git a/xlators/cluster/afr/src/afr-common.c b/xlators/cluster/afr/src/afr-common.c index 576199db137..dae1b9d8a48 100644 --- a/xlators/cluster/afr/src/afr-common.c +++ b/xlators/cluster/afr/src/afr-common.c @@ -3874,7 +3874,7 @@ afr_notify (xlator_t *this, int32_t event, "All subvolumes are down. Going offline " "until atleast one of them comes back up."); } else { - event = GF_EVENT_CHILD_MODIFIED; + event = GF_EVENT_SOME_CHILD_DOWN; } priv->last_event[idx] = event; diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index 6af55d5d42e..982b155d57c 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -7658,6 +7658,12 @@ dht_notify (xlator_t *this, int event, void *data, ...) break; + case GF_EVENT_SOME_CHILD_DOWN: + subvol = data; + propagate = 1; + + break; + case GF_EVENT_CHILD_DOWN: subvol = data; -- cgit