]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: kill request_redirect_t::osd_instructions entirely. 24458/head
authorRadoslaw Zarzynski <rzarzyns@redhat.com>
Sat, 6 Oct 2018 13:54:24 +0000 (15:54 +0200)
committerRadoslaw Zarzynski <rzarzyns@redhat.com>
Sat, 27 Oct 2018 06:19:44 +0000 (08:19 +0200)
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
src/osd/osd_types.cc
src/osd/osd_types.h

index 3fdc2dc32214508d4e0ec906234382d29ef3f54e..70c3273b08ec4fd6b135b7d8c3e8880909606272 100644 (file)
@@ -248,16 +248,21 @@ void request_redirect_t::encode(bufferlist& bl) const
   ENCODE_START(1, 1, bl);
   encode(redirect_locator, bl);
   encode(redirect_object, bl);
-  encode(osd_instructions, bl);
+  // legacy of the removed osd_instructions member
+  encode((uint32_t)0, bl);
   ENCODE_FINISH(bl);
 }
 
 void request_redirect_t::decode(bufferlist::const_iterator& bl)
 {
   DECODE_START(1, bl);
+  uint32_t legacy_osd_instructions_len;
   decode(redirect_locator, bl);
   decode(redirect_object, bl);
-  decode(osd_instructions, bl);
+  decode(legacy_osd_instructions_len, bl);
+  if (legacy_osd_instructions_len) {
+    bl.advance(legacy_osd_instructions_len);
+  }
   DECODE_FINISH(bl);
 }
 
index b7a2e889c94d72be8234ba5aea7097a5fce9e3aa..5d36df4ffb1972923fb6213ba780eb32094772bb 100644 (file)
@@ -289,7 +289,6 @@ struct request_redirect_t {
 private:
   object_locator_t redirect_locator; ///< this is authoritative
   string redirect_object; ///< If non-empty, the request goes to this object name
-  bufferlist osd_instructions; ///< a bufferlist for the OSDs, passed but not interpreted by clients
 
   friend ostream& operator<<(ostream& out, const request_redirect_t& redir);
 public:
@@ -303,8 +302,6 @@ public:
                               const string& robj) :
       redirect_locator(orig), redirect_object(robj) {}
 
-  const bufferlist& get_instructions() { return osd_instructions; }
-
   bool empty() const { return redirect_locator.empty() &&
                              redirect_object.empty(); }