SERVER-35706 ctor race in LogicalSessionCacheImpl

The constructor for LogicalSessionCacheImpl writes to fields after it
has kicked off periodic jobs that also write to those fields.  It
doesn't do it for any good reason, so re-order to fix the race.

1 files changed, 3 insertions, 2 deletions

diff --git a/src/mongo/db/logical_session_cache_impl.cpp b/src/mongo/db/logical_session_cache_impl.cpp
index a6f1b2b7d45..3c1082dbc35 100644
--- a/src/mongo/db/logical_session_cache_impl.cpp
+++ b/src/mongo/db/logical_session_cache_impl.cpp
@@ -64,6 +64,9 @@ LogicalSessionCacheImpl::LogicalSessionCacheImpl(
       _service(std::move(service)),
       _sessionsColl(std::move(collection)),
       _transactionReaper(std::move(transactionReaper)) {
+    _stats.setLastSessionsCollectionJobTimestamp(now());
+    _stats.setLastTransactionReaperJobTimestamp(now());
+
     if (!disableLogicalSessionCacheRefresh) {
         _service->scheduleJob({"LogicalSessionCacheRefresh",
                                [this](Client* client) { _periodicRefresh(client); },
@@ -74,8 +77,6 @@ LogicalSessionCacheImpl::LogicalSessionCacheImpl(
                                    _refreshInterval});
         }
     }
-    _stats.setLastSessionsCollectionJobTimestamp(now());
-    _stats.setLastTransactionReaperJobTimestamp(now());
 }
 
 LogicalSessionCacheImpl::~LogicalSessionCacheImpl() {