From: Joao Eduardo Luis Date: Tue, 23 Jul 2013 15:34:26 +0000 (+0100) Subject: mon: OSDMonitor: get rid of encode_full() as we don't use it. X-Git-Tag: v0.67-rc2~18^2~1 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=bc8d62fe31a1ab7177cb7268dd4652fbeddde5a3;p=ceph.git mon: OSDMonitor: get rid of encode_full() as we don't use it. We have delegated this to encode_trim_extra() since 7fb3804fb860dcd0340dd3f7c39eec4315f8e4b6 -- no need to keep this code around. Signed-off-by: Joao Eduardo Luis Reviewed-by: Sage Weil --- diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc index a8396bf696b12..8e5eed13f5929 100644 --- a/src/mon/OSDMonitor.cc +++ b/src/mon/OSDMonitor.cc @@ -526,17 +526,6 @@ void OSDMonitor::encode_pending(MonitorDBStore::Transaction *t) put_last_committed(t, pending_inc.epoch); } -void OSDMonitor::encode_full(MonitorDBStore::Transaction *t) -{ - dout(10) << __func__ << " osdmap e " << osdmap.epoch << dendl; - assert(get_last_committed() == osdmap.epoch); - - bufferlist osdmap_bl; - osdmap.encode(osdmap_bl); - put_version_full(t, osdmap.epoch, osdmap_bl); - put_version_latest_full(t, osdmap.epoch); -} - void OSDMonitor::share_map_with_random_osd() { if (osdmap.get_num_up_osds() == 0) { diff --git a/src/mon/OSDMonitor.h b/src/mon/OSDMonitor.h index d7cb8fdf369a5..f66c676b53d91 100644 --- a/src/mon/OSDMonitor.h +++ b/src/mon/OSDMonitor.h @@ -150,10 +150,14 @@ private: void update_from_paxos(bool *need_bootstrap); void create_pending(); // prepare a new pending void encode_pending(MonitorDBStore::Transaction *t); - virtual void encode_full(MonitorDBStore::Transaction *t); void on_active(); void on_shutdown(); + /** + * we haven't delegated full version stashing to paxosservice for some time + * now, making this function useless in current context. + */ + virtual void encode_full(MonitorDBStore::Transaction *t) { } /** * do not let paxosservice periodically stash full osdmaps, or we will break our * locally-managed full maps. (update_from_paxos loads the latest and writes them