summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSage Weil <sage@inktank.com>2013-05-31 14:30:48 -0700
committerSage Weil <sage@inktank.com>2013-06-17 14:24:44 -0700
commit65e63ba252f3ce336485b066f2d0367331280325 (patch)
tree6de103e52b827238a21b440b15ee542811b78b41
parentacae697cc36edc5f7db76616a59f263531adee17 (diff)
downloadceph-65e63ba252f3ce336485b066f2d0367331280325.tar.gz
mon: no need to refresh from _active
The refresh is done explicitly by the monitor, independent of the more fragile PaxosService callbacks. Signed-off-by: Sage Weil <sage@inktank.com>
-rw-r--r--src/mon/PaxosService.cc10
1 files changed, 4 insertions, 6 deletions
diff --git a/src/mon/PaxosService.cc b/src/mon/PaxosService.cc
index 71c1ea52790..8dc1b42bec5 100644
--- a/src/mon/PaxosService.cc
+++ b/src/mon/PaxosService.cc
@@ -24,11 +24,12 @@
#define dout_subsys ceph_subsys_paxos
#undef dout_prefix
-#define dout_prefix _prefix(_dout, mon, paxos, service_name)
-static ostream& _prefix(std::ostream *_dout, Monitor *mon, Paxos *paxos, string service_name) {
+#define dout_prefix _prefix(_dout, mon, paxos, service_name, get_first_committed(), get_last_committed())
+static ostream& _prefix(std::ostream *_dout, Monitor *mon, Paxos *paxos, string service_name,
+ version_t fc, version_t lc) {
return *_dout << "mon." << mon->name << "@" << mon->rank
<< "(" << mon->get_state_name()
- << ").paxosservice(" << service_name << ") ";
+ << ").paxosservice(" << service_name << " " << fc << ".." << lc << ") ";
}
bool PaxosService::dispatch(PaxosServiceMessage *m)
@@ -260,9 +261,6 @@ void PaxosService::_active()
}
dout(10) << "_active" << dendl;
- // pull latest from paxos
- refresh();
-
scrub();
// create pending state?