fix(state): Updates `total_size_on_disk` metric to be the total sst file size - Githubissues

ZcashFoundation / zebra

Zcash - Financial Privacy in Rust 🦓

https://zfnd.org/zebra/

Apache License 2.0

412 stars 106 forks source link

fix(state): Updates `total_size_on_disk` metric to be the total sst file size #8966

Closed arya2 closed 1 month ago

arya2 commented 1 month ago

Motivation

The db metrics printed by Zebra when it starts up seem too high, it looks like the total sst size in RocksDB includes the live database size.

Solution

Avoid adding the live database size to the total db size when printing db metrics

Tests

Manually tested that the output matches the cache directory size reported by du -hs

PR Author's Checklist

[x] The PR name will make sense to users.
[x] The solution is tested.
[x] The documentation is up to date.
[x] The PR has a priority label.

PR Reviewer's Checklist

[ ] The PR Author's checklist is complete.
[ ] The PR resolves the issue.