kvserver: update compacted bytes timeseries to include BlobRewrite bytes

annrpom · annrpom · commit 7f88d98a0140 · 2025-11-10T12:09:56.000-05:00
This patch updates `CompactedBytes()` to include bytes written/read during a blob file rewrite. This function is used to determine our compaction bytes written/read timeseries. Epic: none Fixes: #157024 Release note: None
diff --git a/pkg/storage/engine.go b/pkg/storage/engine.go
@@ -1368,6 +1368,8 @@ func (m *Metrics) CompactedBytes() (read, written uint64) {
 		read += lm.TableBytesRead + lm.BlobBytesRead
 		written += lm.TablesCompacted.Bytes + lm.BlobBytesCompacted
 	}
+	read += uint64(m.Metrics.Compact.BlobFileRewrite.BytesRead)
+	written += uint64(m.Metrics.Compact.BlobFileRewrite.BytesWritten)
 	return read, written
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1368,6 +1368,8 @@ func (m *Metrics) CompactedBytes() (read, written uint64) {`
`1368`	`1368`	`read += lm.TableBytesRead + lm.BlobBytesRead`
`1369`	`1369`	`written += lm.TablesCompacted.Bytes + lm.BlobBytesCompacted`
`1370`	`1370`	`}`
	`1371`	`+ read += uint64(m.Metrics.Compact.BlobFileRewrite.BytesRead)`
	`1372`	`+ written += uint64(m.Metrics.Compact.BlobFileRewrite.BytesWritten)`
`1371`	`1373`	`return read, written`
`1372`	`1374`	`}`
`1373`	`1375`