diff options
| author | Richard Wareing <rwareing@fb.com> | 2015-11-30 17:45:30 -0800 |
|---|---|---|
| committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-08-28 16:39:28 +0000 |
| commit | af38bab4e6185f97f84fbfe096d85d8ec4727846 (patch) | |
| tree | cf0239f3e3b7191b84425d5b6ff494c48701d083 /cli/src/cli-cmd-snapshot.c | |
| parent | da0c8c10dc693f75524d6d9f71af87db98f4eda0 (diff) | |
cluster/afr: Remove "compatability" code from SHD flow
Summary:
- After examining some lock revocations, I recommend we remove
this (problematic?) code which causes a lock to be held outside of the
SHD locking domain.
- The theory is that this lock should never conflict with anything
outside of the SHD flow since the offsets are so huge. However in
practice with lock revocation the lock lives a very long time, and I
worry what the other implications of this might be.
Test Plan: - Run tests/basic/afr/* (w/ D2706710)
Reviewers: dph, moox, sshreyas
Reviewed By: sshreyas
Differential Revision: https://phabricator.fb.com/D2706717
Change-Id: I1f358f66810d104f28def9d1ac2a4fde3d073c92
Signed-off-by: Jeff Darcy <jdarcy@fb.com>
Reviewed-on: https://review.gluster.org/18123
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Jeff Darcy <jeff@pl.atyp.us>
Tested-by: Jeff Darcy <jeff@pl.atyp.us>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'cli/src/cli-cmd-snapshot.c')
0 files changed, 0 insertions, 0 deletions