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

ethdb/leveldb: update leveldb metrics collection #27643

Merged
merged 4 commits into from
Jul 5, 2023

Conversation

Exca-DK
Copy link
Contributor

@Exca-DK Exca-DK commented Jul 3, 2023

This PR implements feature described in #27497
Additionally, it includes a small refactor of the meter method to be more future-proof.

@rjl493456442
Copy link
Member

rjl493456442 commented Jul 4, 2023

@fjl fjl changed the title ethdb/leveldb: include levels gauge ethdb/leveldb: update leveldb metrics collection Jul 5, 2023
@fjl fjl added this to the 1.12.1 milestone Jul 5, 2023
@fjl fjl merged commit 863f6da into ethereum:master Jul 5, 2023
devopsbo3 pushed a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
This removes text parsing in leveldb metrics collection code. All metrics
can now be accessed through the stats API provided by leveldb.

We also add new gauge-typed metrics that count the number of tables at each level.

---------

Co-authored-by: Exca-DK <Exca-DK@users.noreply.github.com>
Co-authored-by: Gary Rong <garyrong0905@gmail.com>
Co-authored-by: Felix Lange <fjl@twurst.com>
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants