diff options
author | Venky Shankar <vshankar@redhat.com> | 2014-07-18 15:36:42 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2014-07-23 16:15:21 -0700 |
commit | 8a24b1487d440c23b00b56aba5b9bfb77af6ef03 (patch) | |
tree | f71ac65c27cc56a2a3c95f3ce63d79c13af36dea /geo-replication | |
parent | 1706151a42ce92057b01f4c6869bf66c65769021 (diff) |
features/changelog: Capture "correct" internal FOPs
This patch fixes changelog capturing internal FOPs in a cascaded
setup, where the intermediate master would record internal FOPs
(generated by DHT on link()/rename()). This is due to I/O happening
on the intermediate slave on geo-replication's auxillary mount with
client-pid -1. Currently, the internal FOP capturing logic depends
on client pid being non-negative and the presence of a special key
in dictionary. Due to this, internal FOPs on an inter-mediate master
would be recorded in the changelog. Checking client-pid being
non-negative was introduced to capture AFR self-heal traffic in
changelog, thereby breaking cascading setups. By coincidence,
AFR self-heal daemon uses -1 as frame->root->pid thereby making
is hard to differentiate b/w geo-rep's auxillary mount and self-heal
daemon.
Change-Id: Ib7bd71e80dd1856770391edb621ba9819cab7056
BUG: 1122037
Original-Author: Venky Shankar <vshankar@redhat.com>
Signed-off-by: Kotresh H R <khiremat@redhat.com>
Reviewed-on: http://review.gluster.org/8347
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'geo-replication')
0 files changed, 0 insertions, 0 deletions