Procházet zdrojové kódy

Merge pull request #28640 from overleaf/bg-fix-chunk-store-metrics

update metrics for getChangesSinceVersion to include status and hasMore flag

GitOrigin-RevId: 389353425a12d6d7c92aa3ae37d6553fc1c7fc1d
Brian Gough před 1 rokem
rodič
revize
ff267ca246

+ 5 - 1
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 }
   }