From ff267ca24622414183310da6876b711e3b4881f6 Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Mon, 22 Sep 2025 16:20:29 +0100 Subject: [PATCH] Merge pull request #28640 from overleaf/bg-fix-chunk-store-metrics update metrics for getChangesSinceVersion to include status and hasMore flag GitOrigin-RevId: 389353425a12d6d7c92aa3ae37d6553fc1c7fc1d --- services/history-v1/storage/lib/chunk_store/index.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/services/history-v1/storage/lib/chunk_store/index.js b/services/history-v1/storage/lib/chunk_store/index.js index ca46d85964..d860a9b88a 100644 --- a/services/history-v1/storage/lib/chunk_store/index.js +++ b/services/history-v1/storage/lib/chunk_store/index.js @@ -246,7 +246,11 @@ async function getChangesSinceVersion(projectId, since) { const result = await redisBackend.getChangesSinceVersion(projectId, since) if (result.status === 'ok') { // Successfully got changes from Redis, no more changes available beyond what Redis has - metrics.inc('chunk_store.get_changes_since_version', 1, { source: 'redis' }) + metrics.inc('chunk_store.get_changes_since_version', 1, { + source: 'redis', + hasMore: 'false', + status: result.status, + }) return { changes: result.changes || [], hasMore: false } }