diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2015-07-26 11:19:28 +0530 |
---|---|---|
committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2015-08-31 22:56:29 -0700 |
commit | 9f2a399055d3738457dbc461b1e9e99f718884e0 (patch) | |
tree | 8d00152e03fd432b7a9169ef3c4b4ec54cdbe309 /xlators/cluster/afr/src/afr.h | |
parent | 93519aed0f32570bd6fb8e0113f76a75545ab804 (diff) |
cluster/afr: Make [f]xattrop metadata transaction
Backport of http://review.gluster.com/11809
Problem:
When xlators above afr do [f]xattrop when one of the bricks is down, after the
brick comes backup, the metadata is not healed because [f]xattrop is not
considered a transaction.
Fix:
Treat [f]xattrop as transaction so that changes done by xlators above afr are
marked for heal when some of the bricks were down at the time of [f]xattrop.
BUG: 1248890
Change-Id: Ibe69aa0ca6be9b4b4134dc2879b306e2e9c4cde8
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/11810
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Ravishankar N <ravishankar@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr.h')
-rw-r--r-- | xlators/cluster/afr/src/afr.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/xlators/cluster/afr/src/afr.h b/xlators/cluster/afr/src/afr.h index 2c225cd6462..008839a7312 100644 --- a/xlators/cluster/afr/src/afr.h +++ b/xlators/cluster/afr/src/afr.h @@ -262,6 +262,7 @@ struct afr_reply { int valid; int32_t op_ret; int32_t op_errno; + dict_t *xattr;/*For xattrop*/ dict_t *xdata; struct iatt poststat; struct iatt postparent; @@ -568,12 +569,9 @@ typedef struct _afr_local { struct { dict_t *xattr; + gf_xattrop_flags_t optype; } xattrop; - struct { - dict_t *xattr; - } fxattrop; - /* dir write */ struct { @@ -734,6 +732,8 @@ typedef struct _afr_local { dict_t *xdata_req; dict_t *xdata_rsp; + dict_t *xattr_rsp; /*for [f]xattrop*/ + mode_t umask; int xflag; gf_boolean_t do_discovery; |