Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(metrics): Use correct Badger metrics. #239

Merged
merged 2 commits into from
Jun 4, 2021

Conversation

danielmai
Copy link
Contributor

This PR reflects the update to Dgraph metrics made in hypermodeinc/dgraph#7507:

This renames the Badger metrics in the /metrics endpoint to remove the
Badger version in the metric names. This allows the metric name to
remain the same across Badger major versions.

Previous Name New Name
badger_v3_disk_reads_total badger_disk_reads_total
badger_v3_disk_writes_total badger_disk_writes_total
badger_v3_read_bytes badger_read_bytes
badger_v3_written_bytes badger_written_bytes
badger_v3_lsm_bloom_hits_total badger_lsm_bloom_hits_total
badger_v3_gets_total badger_gets_total
badger_v3_puts_total badger_puts_total
badger_v3_memtable_gets_total badger_memtable_gets_total
badger_v3_lsm_size_bytes badger_lsm_size_bytes
badger_v3_vlog_size_bytes badger_vlog_size_bytes

This doc change needs to be cherry-picked to release/v21.03.

@netlify
Copy link

netlify bot commented Jun 4, 2021

✔️ Deploy Preview for dgraph-docs-repo ready!

🔨 Explore the source changes: 7754be0

🔍 Inspect the deploy log: https://app.netlify.com/sites/dgraph-docs-repo/deploys/60b9c0d788796600071caf3a

😎 Browse the preview: https://deploy-preview-239--dgraph-docs-repo.netlify.app

@danielmai danielmai merged commit faef8de into master Jun 4, 2021
@danielmai danielmai deleted the danielmai/metrics-badger branch June 4, 2021 21:57
MichelDiz pushed a commit that referenced this pull request Dec 15, 2022
This PR reflects the update to Dgraph metrics made in hypermodeinc/dgraph#7507
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants