diff options
| author | Pranith Kumar K <pkarampu@redhat.com> | 2014-04-21 03:13:58 +0530 |
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2014-05-10 13:30:48 -0700 |
| commit | 46bf591eb87d743baacd30a85b99fd47db4fb055 (patch) | |
| tree | 20ba994c07283e4d91d45f68ef844614da3045e3 /xlators/cluster/afr/src/afr-dir-write.c | |
| parent | ba328340e878c3156418bb3443c35a7db02a0f4b (diff) | |
cluster/afr: Fix bugs in quorum implementation
- Have common place to perform quorum fop wind check
- Check if fop succeeded in a way that matches quorum
to avoid marking changelog in split-brain.
Change-Id: I663072ece0e1de6e1ee9fccb03e1b6c968793bc5
BUG: 1066996
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/7513
Reviewed-by: Ravishankar N <ravishankar@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr-dir-write.c')
| -rw-r--r-- | xlators/cluster/afr/src/afr-dir-write.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/xlators/cluster/afr/src/afr-dir-write.c b/xlators/cluster/afr/src/afr-dir-write.c index 1943b719bb5..3cdec64536d 100644 --- a/xlators/cluster/afr/src/afr-dir-write.c +++ b/xlators/cluster/afr/src/afr-dir-write.c @@ -432,8 +432,6 @@ afr_create (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(create,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -637,8 +635,6 @@ afr_mknod (call_frame_t *frame, xlator_t *this, loc_t *loc, mode_t mode, priv = this->private; - QUORUM_CHECK(mknod,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -842,8 +838,6 @@ afr_mkdir (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(mkdir,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -1046,8 +1040,6 @@ afr_link (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(link,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -1249,8 +1241,6 @@ afr_symlink (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(symlink,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -1464,8 +1454,6 @@ afr_rename (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(rename,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -1685,8 +1673,6 @@ afr_unlink (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(unlink,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; @@ -1889,8 +1875,6 @@ afr_rmdir (call_frame_t *frame, xlator_t *this, priv = this->private; - QUORUM_CHECK(rmdir,out); - transaction_frame = copy_frame (frame); if (!transaction_frame) { op_errno = ENOMEM; |