]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
messages/MOSDOpReply: remove misleading may_read/may_write
authorSage Weil <sage@inktank.com>
Fri, 7 Dec 2012 21:18:50 +0000 (13:18 -0800)
committerSage Weil <sage@inktank.com>
Thu, 27 Dec 2012 20:12:40 +0000 (12:12 -0800)
These are OpRequest properties, calculated/enforced at the OSD.  They don't
belong in the MOSDOp or MOSDOpReply messages.

Signed-off-by: Sage Weil <sage@inktank.com>
src/messages/MOSDOpReply.h

index 9f47e5db95c999441b5f61d603174e6cbda6b518..65f7b23987cde710601501b8845fa3ac6f40ad4c 100644 (file)
@@ -54,9 +54,6 @@ public:
   int get_result() const { return result; }
   eversion_t get_version() { return reassert_version; }
   
-  bool may_read() const { return flags & CEPH_OSD_FLAG_READ; }
-  bool may_write() const { return flags & CEPH_OSD_FLAG_WRITE; }
-
   void set_result(int r) { result = r; }
   void set_version(eversion_t v) { reassert_version = v; }
 
@@ -200,14 +197,12 @@ public:
   void print(ostream& out) const {
     out << "osd_op_reply(" << get_tid()
        << " " << oid << " " << ops;
-    if (may_write()) {
-      if (is_ondisk())
-       out << " ondisk";
-      else if (is_onnvram())
-       out << " onnvram";
-      else
-       out << " ack";
-    }
+    if (is_ondisk())
+      out << " ondisk";
+    else if (is_onnvram())
+      out << " onnvram";
+    else
+      out << " ack";
     out << " = " << get_result();
     if (get_result() < 0) {
       char buf[80];