Merge branch 'master' into fix_tck

vesoft-inc · Apr 15, 2022 · a25bc9c · a25bc9c
2 parents 8cc4a43 + 365ad83
commit a25bc9c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/src/graph/service/GraphService.cpp b/src/graph/service/GraphService.cpp
@@ -125,7 +125,6 @@ folly::Future<AuthResponse> GraphService::future_authenticate(const std::string&
 void GraphService::signout(int64_t sessionId) {
   VLOG(2) << "Sign out session " << sessionId;
   sessionManager_->removeSession(sessionId);
-  stats::StatsManager::decValue(kNumActiveSessions);
 }
 
 folly::Future<ExecutionResponse> GraphService::future_executeWithParameter(

diff --git a/src/graph/session/GraphSessionManager.cpp b/src/graph/session/GraphSessionManager.cpp
@@ -180,6 +180,7 @@ void GraphSessionManager::removeSession(SessionID id) {
   auto sessionCopy = iter->second->getSession();
   std::string key = sessionCopy.get_user_name() + sessionCopy.get_client_ip();
   activeSessions_.erase(iter);
+  stats::StatsManager::decValue(kNumActiveSessions);
   // delete session count from cache
   subSessionCount(key);
 }