From 93e7c0aa6730f420da50ccec9ea572e4ebd8828b Mon Sep 17 00:00:00 2001
From: Ivan Savenko <saven.ivan@gmail.com>
Date: Sun, 10 May 2026 01:07:18 +0300
Subject: [PATCH] Optimize getActiveGameRooms call

This call currently takes large part of DB access time.

According to EXPLAIN QUERY PLAN it looks like WHERE xxx IN(1,2,3) causes
SQLite to use full table SCAN instead of using available index.

For getActiveGameRooms this change decreases query time from ~60-70 ms
to <1 ms
---
 lobby/LobbyDatabase.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lobby/LobbyDatabase.cpp b/lobby/LobbyDatabase.cpp
index 51e08abde..922573cd5 100644
--- a/lobby/LobbyDatabase.cpp
+++ b/lobby/LobbyDatabase.cpp
@@ -156,7 +156,7 @@ void LobbyDatabase::prepareStatements()
 		SELECT grp.roomID
 		FROM gameRoomPlayers grp
 		LEFT JOIN gameRooms gr ON gr.roomID = grp.roomID
-		WHERE accountID = ? AND status IN (1, 2, 3)
+		WHERE accountID = ? AND status >= 1 AND status <= 3
 		LIMIT 1
 	)");
 
@@ -170,7 +170,7 @@ void LobbyDatabase::prepareStatements()
 		SELECT roomID, hostAccountID, displayName, description, status, playerLimit, version, strftime('%s',CURRENT_TIMESTAMP)- strftime('%s',gr.creationTime)  AS secondsElapsed
 		FROM gameRooms gr
 		LEFT JOIN accounts a ON gr.hostAccountID = a.accountID
-		WHERE status IN (1, 2, 3)
+		WHERE status >= 1 AND status <= 3
 		ORDER BY secondsElapsed ASC
 	)");
 
@@ -275,7 +275,7 @@ void LobbyDatabase::prepareStatements()
 		SELECT COUNT(accountID)
 		FROM gameRoomPlayers grp
 		LEFT JOIN gameRooms gr ON gr.roomID = grp.roomID
-		WHERE accountID = ? AND status IN (1, 2, 3)
+		WHERE accountID = ? AND status >= 1 AND status <= 3
 	)");
 
 	isAccountIDExistsStatement = database->prepare(R"(