From 78fd49ed3a22a976112973e31883ccb48ed8efe8 Mon Sep 17 00:00:00 2001
From: Rene Cannao <rene@proxysql.com>
Date: Thu, 25 Jan 2024 03:27:00 +0000
Subject: [PATCH] Minor code optimization

---
 include/MySQL_Session.h | 1 +
 lib/MySQL_Session.cpp   | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/include/MySQL_Session.h b/include/MySQL_Session.h
index 5e0752fecc..6c08e4ac1e 100644
--- a/include/MySQL_Session.h
+++ b/include/MySQL_Session.h
@@ -261,6 +261,7 @@ class MySQL_Session
 	 *  maintenance thread. These values will be used to release the retained connections in the specific
 	 *  hostgroups in housekeeping operations, before client packet processing. Currently 'housekeeping_before_pkts'.
 	 */
+	size_t hgs_expired_conns_cnt = 0;
 	std::vector<int32_t> hgs_expired_conns {};
 	char * default_schema;
 	char * user_attributes;
diff --git a/lib/MySQL_Session.cpp b/lib/MySQL_Session.cpp
index a0878dcf98..612eaf70e5 100644
--- a/lib/MySQL_Session.cpp
+++ b/lib/MySQL_Session.cpp
@@ -770,6 +770,7 @@ void MySQL_Session::update_expired_conns(const vector<function<bool(MySQL_Connec
 			}
 		}
 	}
+	hgs_expired_conns_cnt = hgs_expired_conns.size();
 }
 
 MySQL_Backend * MySQL_Session::create_backend(int hostgroup_id, MySQL_Data_Stream *_myds) {
@@ -4837,6 +4838,7 @@ void MySQL_Session::housekeeping_before_pkts() {
 		if (hgs_expired_conns.empty() == false) {
 			hgs_expired_conns.clear();
 		}
+		hgs_expired_conns_cnt = hgs_expired_conns.size();
 	}
 }
 
@@ -4884,7 +4886,9 @@ int MySQL_Session::handler() {
 	}
 	}
 
-	housekeeping_before_pkts();
+	if (unlikely(hgs_expired_conns_cnt != 0)) {
+		housekeeping_before_pkts();
+	}
 	handler_ret = get_pkts_from_client(wrong_pass, pkt);
 	if (handler_ret != 0) {
 		return handler_ret;