fix comment

vesoft-inc · Feb 14, 2022 · 0f1554f · 0f1554f
1 parent 59f20c9
commit 0f1554f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/graph/service/GraphService.cpp b/src/graph/service/GraphService.cpp
@@ -80,7 +80,7 @@ folly::Future<AuthResponse> GraphService::future_authenticate(const std::string&
     return future;
   }
 
-  if (!sessionManager_->isOutOfConnections()) {
+  if (sessionManager_->isOutOfConnections()) {
     ctx->resp().errorCode = ErrorCode::E_TOO_MANY_CONNECTIONS;
     ctx->resp().errorMsg.reset(new std::string("Too many connections in the cluster"));
     ctx->finish();

diff --git a/src/graph/session/GraphSessionManager.h b/src/graph/session/GraphSessionManager.h
@@ -48,15 +48,15 @@ class GraphSessionManager final : public SessionManager<ClientSession> {
   folly::Future<StatusOr<std::shared_ptr<ClientSession>>> createSession(
       const std::string userName, const std::string clientIp, folly::Executor* runner) override;
 
-  // Whether not exceeds the max allowed connections.
+  // Whether exceeds the max allowed connections.
   bool isOutOfConnections() {
     if (activeSessions_.size() >= static_cast<uint64_t>(FLAGS_max_allowed_connections)) {
       LOG(INFO) << "The sessions of the cluster has more than "
                    "max_allowed_connections: "
                 << FLAGS_max_allowed_connections;
-      return false;
+      return true;
     }
-    return true;
+    return false;
   }
 
   // Removes a session from both local and meta server.