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 <joao.luis@inktank.com>

2 files changed, 5 insertions, 12 deletions

diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index a8396bf696b..8e5eed13f59 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 d7cb8fdf369..f66c676b53d 100644
--- a/src/mon/OSDMonitor.h
+++ b/src/mon/OSDMonitor.h
@@ -150,11 +150,15 @@ 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
    * out going forward from there, but if we just synced that may mean we skip some.)