]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
librados: OPERATION_BALANCE/LOCALIZE_READS comment is outdated
authorIlya Dryomov <idryomov@gmail.com>
Wed, 6 Oct 2021 21:14:22 +0000 (23:14 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Wed, 6 Oct 2021 21:14:52 +0000 (23:14 +0200)
With the introduction of min_last_complete_ondisk on the OSD side,
it is safe to use these for non-immutable data.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
src/include/rados/librados.hpp

index 76fe6dfd0cd69181955d027de17bf90d4794188c..7c883c33648d008f33088bd9bc18d70cd37ea186 100644 (file)
@@ -246,9 +246,11 @@ inline namespace v14_2_0 {
   /**
    * These flags apply to the ObjectOperation as a whole.
    *
-   * BALANCE_READS and LOCALIZE_READS should only be used
-   * when reading from data you're certain won't change,
-   * like a snapshot, or where eventual consistency is ok.
+   * Prior to octopus BALANCE_READS and LOCALIZE_READS should only
+   * be used when reading from data you're certain won't change, like
+   * a snapshot, or where eventual consistency is ok.  Since octopus
+   * (get_min_compatible_osd() >= CEPH_RELEASE_OCTOPUS) both are safe
+   * for general use.
    *
    * ORDER_READS_WRITES will order reads the same way writes are
    * ordered (e.g., waiting for degraded objects).  In particular, it