summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/mongo/db/logical_session_cache_impl.cpp13
1 files changed, 12 insertions, 1 deletions
diff --git a/src/mongo/db/logical_session_cache_impl.cpp b/src/mongo/db/logical_session_cache_impl.cpp
index 2f5a2731ea4..2815c075a40 100644
--- a/src/mongo/db/logical_session_cache_impl.cpp
+++ b/src/mongo/db/logical_session_cache_impl.cpp
@@ -344,13 +344,24 @@ void LogicalSessionCacheImpl::_refresh(Client* client) {
_stats.setLastSessionsCollectionJobEntriesEnded(explicitlyEndingSessions.size());
}
-
// Find which running, but not recently active sessions, are expired, and add them
// to the list of sessions to kill cursors for
KillAllSessionsByPatternSet patterns;
auto openCursorSessions = _service->getOpenCursorSessions();
+ // Exclude sessions added to _activeSessions from the openCursorSession to avoid race between
+ // killing cursors on the removed sessions and creating sessions.
+ {
+ stdx::lock_guard<stdx::mutex> lk(_cacheMutex);
+
+ for (const auto& it : _activeSessions) {
+ auto newSessionIt = openCursorSessions.find(it.first);
+ if (newSessionIt != openCursorSessions.end()) {
+ openCursorSessions.erase(newSessionIt);
+ }
+ }
+ }
// think about pruning ending and active out of openCursorSessions
auto statusAndRemovedSessions = _sessionsColl->findRemovedSessions(opCtx, openCursorSessions);