crimson/os: alias errorator for CyanStore::get_attrs().

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

diff --git a/src/crimson/os/cyan_store.cc b/src/crimson/os/cyan_store.cc
index 0dafec55cef72487e668990cf9ec4e702c5baca1..e8088ff65ef7c8c3d1defb142d2580514f9f7ae3 100644 (file)
--- a/src/crimson/os/cyan_store.cc
+++ b/src/crimson/os/cyan_store.cc
@@ -198,11 +198,10 @@ seastar::future<ceph::bufferlist> CyanStore::read(CollectionRef ch,
   return seastar::make_ready_future<ceph::bufferlist>(o->read(offset, l));
 }
 
-crimson::errorator<crimson::ct_error::enoent,
-                   crimson::ct_error::enodata>::future<ceph::bufferptr>
-CyanStore::get_attr(CollectionRef ch,
-                    const ghobject_t& oid,
-                    std::string_view name) const
+CyanStore::get_attr_errorator::future<ceph::bufferptr> CyanStore::get_attr(
+  CollectionRef ch,
+  const ghobject_t& oid,
+  std::string_view name) const
 {
   auto c = static_cast<Collection*>(ch.get());
   logger().debug("{} {} {}",
@@ -212,8 +211,7 @@ CyanStore::get_attr(CollectionRef ch,
     return crimson::make_error<crimson::ct_error::enoent>();
   }
   if (auto found = o->xattr.find(name); found != o->xattr.end()) {
-    return crimson::errorator<crimson::ct_error::enoent,
-                              crimson::ct_error::enodata>::its_error_free(
+    return get_attr_errorator::its_error_free(
       seastar::make_ready_future<ceph::bufferptr>(found->second));
   } else {
     return crimson::make_error<crimson::ct_error::enodata>();

diff --git a/src/crimson/os/cyan_store.h b/src/crimson/os/cyan_store.h
index 84311882e52b99bb6920dd442a7091de5bdf1f0b..96f3eb84a231e428513ccd1cfa411beae9c6bf4c 100644 (file)
--- a/src/crimson/os/cyan_store.h
+++ b/src/crimson/os/cyan_store.h
@@ -49,11 +49,10 @@ public:
                                   uint64_t offset,
                                   size_t len,
                                   uint32_t op_flags = 0) final;
-  virtual crimson::errorator<crimson::ct_error::enoent,
-                             crimson::ct_error::enodata>::future<ceph::bufferptr>
-  get_attr(CollectionRef c,
-           const ghobject_t& oid,
-           std::string_view name) const final;
+  get_attr_errorator::future<ceph::bufferptr> get_attr(
+    CollectionRef c,
+    const ghobject_t& oid,
+    std::string_view name) const final;
   seastar::future<attrs_t> get_attrs(CollectionRef c,
                                      const ghobject_t& oid) final;
 

diff --git a/src/crimson/os/futurized_store.h b/src/crimson/os/futurized_store.h
index b7045a7b00ecbe1f74e287be6f4bc5d5b0c033e2..401f9d456d183806cc2aea685492bdb028f17cbc 100644 (file)
--- a/src/crimson/os/futurized_store.h
+++ b/src/crimson/os/futurized_store.h
@@ -47,9 +47,13 @@ public:
                                   uint64_t offset,
                                   size_t len,
                                   uint32_t op_flags = 0) = 0;
-  virtual crimson::errorator<crimson::ct_error::enoent,
-                             crimson::ct_error::enodata>::future<ceph::bufferptr>
-  get_attr(CollectionRef c, const ghobject_t& oid, std::string_view name) const = 0;
+  using get_attr_errorator = crimson::errorator<
+    crimson::ct_error::enoent,
+    crimson::ct_error::enodata>;
+  virtual get_attr_errorator::future<ceph::bufferptr> get_attr(
+    CollectionRef c,
+    const ghobject_t& oid,
+    std::string_view name) const = 0;
 
   using attrs_t = std::map<std::string, ceph::bufferptr, std::less<>>;
   virtual seastar::future<attrs_t> get_attrs(CollectionRef c,

diff --git a/src/crimson/osd/pg_backend.cc b/src/crimson/osd/pg_backend.cc
index be8a2f8cfdad524f48971514f1bd2d7714f3f2a9..58c087678eb5f70a3db565a9c534d9b654041c98 100644 (file)
--- a/src/crimson/osd/pg_backend.cc
+++ b/src/crimson/osd/pg_backend.cc
@@ -498,9 +498,7 @@ seastar::future<> PGBackend::getxattr(
   //ctx->delta_stats.num_rd++;
 }
 
-crimson::errorator<crimson::ct_error::enoent,
-                   crimson::ct_error::enodata>::future<ceph::bufferptr>
-PGBackend::getxattr(
+PGBackend::get_attr_errorator::future<ceph::bufferptr> PGBackend::getxattr(
   const hobject_t& soid,
   std::string_view key) const
 {

diff --git a/src/crimson/osd/pg_backend.h b/src/crimson/osd/pg_backend.h
index e831fcfe9c7f5de39d0d40323837fa342b341ad4..6973236bcd894d6d0a5432d7150a43deb0c0452d 100644 (file)
--- a/src/crimson/osd/pg_backend.h
+++ b/src/crimson/osd/pg_backend.h
@@ -87,8 +87,8 @@ public:
   seastar::future<> getxattr(
     const ObjectState& os,
     OSDOp& osd_op) const;
-  crimson::errorator<crimson::ct_error::enoent,
-                     crimson::ct_error::enodata>::future<ceph::bufferptr> getxattr(
+  using get_attr_errorator = crimson::os::FuturizedStore::get_attr_errorator;
+  get_attr_errorator::future<ceph::bufferptr> getxattr(
     const hobject_t& soid,
     std::string_view key) const;