Revert "SERVER-47352 Return connections inline for NetworkInterfaceTL"

This reverts commit ab0d153626c24173f0ea9a26610c02ffc51d9420.

1 files changed, 5 insertions, 5 deletions

diff --git a/src/mongo/executor/network_interface_tl.cpp b/src/mongo/executor/network_interface_tl.cpp
index 647fc1bfb36..dc426f37c85 100644
--- a/src/mongo/executor/network_interface_tl.cpp
+++ b/src/mongo/executor/network_interface_tl.cpp
@@ -797,16 +797,16 @@ void NetworkInterfaceTL::RequestState::resolve(Future<RemoteCommandResponse> fut
             .onError([this, anchor = shared_from_this()](Status error) {
                 // The RCRq failed, wrap the error into a RCRsp with the host and duration
                 return RemoteCommandOnAnyResponse(host, std::move(error), stopwatch.elapsed());
-            })
-            .tapAll([ this, anchor = shared_from_this() ](const auto& swr) noexcept {
-                invariant(swr.isOK());
-                returnConnection(swr.getValue().status);
             });
 
     std::move(anyFuture)                                    //
         .thenRunOn(makeGuaranteedExecutor(baton, reactor))  // Switch to the baton/reactor.
         .getAsync([ this, anchor = shared_from_this() ](auto swr) noexcept {
-            auto response = uassertStatusOK(std::move(swr));
+            auto response = uassertStatusOK(swr);
+            auto status = response.status;
+
+            returnConnection(status);
+
             auto commandStatus = getStatusFromCommandResult(response.data);
             // Ignore maxTimeMS expiration errors for hedged reads without triggering the finish
             // line.