Fix too much warning log

JDRaftKeeper · Nov 22, 2023 · d98236b · d98236b
1 parent c947616
commit d98236b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/src/Service/ConnectionHandler.cpp b/src/Service/ConnectionHandler.cpp
@@ -638,7 +638,6 @@ void ConnectionHandler::sendResponse(const Coordination::ZooKeeperResponsePtr &
 {
     LOG_TRACE(log, "Dispatch response {} to conn handler session {}", response->toString(), toHexString(session_id));
 
-    /// TODO should invoked after response sent to client.
     updateStats(response);
 
     /// We do not need send anything for close request to client.

diff --git a/src/Service/RequestProcessor.cpp b/src/Service/RequestProcessor.cpp
@@ -195,12 +195,6 @@ void RequestProcessor::processCommittedRequest(size_t count)
                 my_pending_requests.erase(committed_request.session_id);
             }
 
-            LOG_WARNING(
-                log,
-                "Session {} is not local, maybe it is because of disconnecting. We still should apply the committed(write) "
-                "request",
-                committed_request.session_id);
-
             applyRequest(committed_request);
             committed_queue.pop();
         }
@@ -209,7 +203,7 @@ void RequestProcessor::processCommittedRequest(size_t count)
         {
             if (committed_request.request->getOpNum() == Coordination::OpNum::Auth)
             {
-                LOG_DEBUG(log, "Apply auth request", committed_request.session_id);
+                LOG_DEBUG(log, "Apply auth request {}", toHexString(committed_request.session_id));
                 applyRequest(committed_request);
                 committed_queue.pop();
             }