diff options
| author | Sage Weil <sage@inktank.com> | 2013-06-07 11:14:58 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-06-19 11:27:05 -0700 |
| commit | ec2ea86ed55e00265c2cc5ad0c94460b4c92865c (patch) | |
| tree | 50c9be9200f983222f1ac5bd99b9472fcc3a8e7d | |
| parent | 7b7ea8e30e20704caad9a841332ecb2e39819a41 (diff) | |
| download | ceph-ec2ea86ed55e00265c2cc5ad0c94460b4c92865c.tar.gz | |
mon/Paxos: not readable when LOCKED
If we are re-proposing a previously accepted value from a previous quorum,
we should not consider it readable, because it is possible it was exposed
to clients as committed (2/3 accepted) but not recored to be committed, and
we do not want to expose old state as readable when new state was
previously readable.
Signed-off-by: Sage Weil <sage@inktank.com>
| -rw-r--r-- | src/mon/Paxos.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mon/Paxos.cc b/src/mon/Paxos.cc index 9bd9726745a..67df5fd373b 100644 --- a/src/mon/Paxos.cc +++ b/src/mon/Paxos.cc @@ -1198,6 +1198,7 @@ bool Paxos::is_readable(version_t v) return (mon->is_peon() || mon->is_leader()) && (is_active() || is_updating()) && + !is_locked() && last_committed > 0 && // must have a value (mon->get_quorum().size() == 1 || // alone, or is_lease_valid()); // have lease |