removed possible introduced HA Redis bug; improved comment

This commit is contained in:
Josh Voyles
2026-05-03 09:39:27 -04:00
parent 9bd33072f4
commit 2154811ffb
2 changed files with 2 additions and 2 deletions

View File

@@ -16,6 +16,8 @@ bootstrapVolume();
* Wraps better-sqlite3 Statement to call `finalize()` immediately after
* execution, freeing native sqlite3_stmt memory deterministically instead
* of waiting for GC. Fixes steady off-heap growth under load (#2120).
* WARNING: Finalizes after first execution — incompatible with drizzle's
* reusable .prepare() builders. No such usage exists in this codebase.
*/
function autoFinalizeStatement(
stmt: BetterSqlite3.Statement

View File

@@ -414,8 +414,6 @@ const removeClient = async (
await redisManager.del(
getNodeConnectionsKey(NODE_ID, clientId)
);
// Remove Redis config version key to prevent indefinite accumulation.
await redisManager.del(getConfigVersionKey(clientId));
} catch (error) {
logger.error(
"Failed to remove client from Redis tracking (cleanup will occur on recovery):",