src/ceph/src/doc/mon-wishlist.txt

   1 Monitor Wish List (issue #10509)
   2 ================================
   3
   4 Low-hanging fruit
   5 -----------------
   6
   7 * audit helpers that put() messages but do not get() them.
   8   where possible, get rid of those put().  No one expects helpers to
   9   put() messages and that may lead to double frees. (issue #9378)
  10
  11 Medium complexity
  12 -----------------
  13
  14 * get rid of QuorumServices.  It seemed like a neat idea, but we only have
  15   one or two and they just add complexity and noise. (issue #10506)
  16
  17 Time consuming / complex
  18 ------------------------
  19
  20 * Split the OSDMonitor.cc file into auxiliary files.  This will mean:
  21
  22   1. Logically split subsystems (osd crush, osd pool, ...)
  23   2. Split the big badass functions, especially prepare/process_command()
  24
  25 * Have Tracked Ops on the monitor, similarly to the OSDs. (issue #10507)
  26
  27   1. Instead of passing messages back and forth, we will pass OpRequests
  28   2. We may be able to get() the message when we create the OpRequest and
  29      put() it upon OpRequest destruction.  This will help controlling the
  30      lifespan of messages and reduce leaks.
  31   3. There will be a fair amount of work changing stuff from Messages to
  32      OpRequests, and we will need to make sure that we reach a format that
  33      is easily supported throughout the monitor
  34
  35   Possible format, off the top of my head:
  36
  37     MonOpRequest:
  38
  39       int op = m->get_type();
  40       Message *m = m.get();
  41
  42       template<typename T>
  43       T* get_message() { return (T*)m.get(); }
  44
  45 * Move to Ref'erenced messages instead of pointers all around.  This would
  46   also help with the Tracked Ops thing, as we'd be able to simply ignore all
  47   the get() and put() stuff behind it. (issue #3500)
  48
  49 Delicate / complex
  50 ------------------
  51
  52 * Finer-grained Paxos::is_readable() and/or PaxosService::is_readable()
  53   (issue #10508)
  54
  55   Rationale: a given service S should be able to read its committed state
  56   even though a Paxos proposal is happening, as long as the on-going
  57   proposal is not a value of service S.