From: Kefu Chai <kchai@redhat.com>
Date: Thu, 17 Dec 2020 12:10:09 +0000 (+0800)
Subject: crimson/osd: return plain value instead of future
X-Git-Tag: v16.1.0~202^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=01fb646d0c930ad334f2e0c4a72bc78a665a9dfc;p=ceph.git

crimson/osd: return plain value instead of future

seastar allows us to use plain value instead of a future<> when the
future is available right away.

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/crimson/osd/replicated_recovery_backend.cc b/src/crimson/osd/replicated_recovery_backend.cc
index 657cd77ea0a9..f31536543977 100644
--- a/src/crimson/osd/replicated_recovery_backend.cc
+++ b/src/crimson/osd/replicated_recovery_backend.cc
@@ -650,12 +650,12 @@ seastar::future<bool> ReplicatedRecoveryBackend::_handle_pull_response(
 	pg.get_recovery_handler()->on_local_recover(
 	    pop.soid, recovering.at(pop.soid).pi->recovery_info,
 	    false, *t);
-	return seastar::make_ready_future<bool>(true);
+	return true;
       } else {
         response->soid = pop.soid;
         response->recovery_info = pi.recovery_info;
         response->recovery_progress = pi.recovery_progress;
-        return seastar::make_ready_future<bool>(false);
+        return false;
       }
     });
   });