From bbe74648f9ae227aa7f034a8dc2fe61b48c738fe Mon Sep 17 00:00:00 2001 From: JackyWoo Date: Wed, 22 Nov 2023 18:49:10 +0800 Subject: [PATCH] Fix too much warning log --- src/Service/ConnectionHandler.cpp | 1 - src/Service/RequestProcessor.cpp | 8 +------- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/src/Service/ConnectionHandler.cpp b/src/Service/ConnectionHandler.cpp index 042635fca0..25304bc41c 100644 --- 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 index 49590ef918..049a98d9db 100644 --- 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(); }