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-30 00:45:28 -0700 |
| commit | b0e125e937cbf4bb02baaa799ff4968a4d1cb1d0 (patch) | |
| tree | 0e66f3a507e847279b22cb9d34827e9b970011f0 /xlators/cluster/afr/src/afr.h | |
| parent | d304916ddf3d6848787c3a668cc36e3395b32069 (diff) | |
cluster/afr: Make [f]xattrop metadata transaction
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.
Change-Id: Iea180f9a456509847c3cd8d5d59a0cdc2712d334
BUG: 1248887
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/11809
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Ravishankar N <ravishankar@redhat.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
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 7f995c731d7..3a4a8794698 100644 --- a/xlators/cluster/afr/src/afr.h +++ b/xlators/cluster/afr/src/afr.h @@ -257,6 +257,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; @@ -563,12 +564,9 @@ typedef struct _afr_local { struct { dict_t *xattr; + gf_xattrop_flags_t optype; } xattrop; - struct { - dict_t *xattr; - } fxattrop; - /* dir write */ struct { @@ -729,6 +727,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; |