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

Support to enable rocksdbs statistics #2243

Merged
merged 2 commits into from
Jul 27, 2020

Conversation

chenxu14
Copy link
Contributor

I think there is a need to enable rocksdb's statistics, so we can dump some metrics to the log, and use it to find bottleneck.
We can enable this by specify --rocksdb_statistics_enable=true in nebula-storaged.conf, and declare the stats level through --rocksdb_stats_level(default value is kExceptHistogramOrTimers)

@Shylock-Hg
Copy link
Contributor

If so I think you could export the option stats_dump_period_sec too.

@chenxu14
Copy link
Contributor Author

If so I think you could export the option stats_dump_period_sec too.

I think we can specify this in --rocksdb_db_options

@jude-zhu jude-zhu requested review from critical27 and dangleptr July 23, 2020 09:20
@dutor dutor requested a review from monadbobo July 24, 2020 02:57
Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this contribution.

@chenxu14
Copy link
Contributor Author

Thanks for your advise, can you help to review again.@dutor

@chenxu14 chenxu14 requested a review from dutor July 24, 2020 04:33
dutor
dutor previously approved these changes Jul 24, 2020
Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

dangleptr
dangleptr previously approved these changes Jul 24, 2020
Copy link
Contributor

@dangleptr dangleptr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done.
Thanks for your contribution.

@dangleptr dangleptr added the ready-for-testing PR: ready for the CI test label Jul 24, 2020
@dangleptr
Copy link
Contributor

Please check your code style with cpplint.
Thanks @chenxu14

@chenxu14 chenxu14 dismissed stale reviews from dangleptr and dutor via e9ff598 July 24, 2020 10:34
@chenxu14 chenxu14 force-pushed the statistics-enable branch from 937fb2a to e9ff598 Compare July 24, 2020 10:34
@chenxu14
Copy link
Contributor Author

Please check your code style with cpplint.
Thanks @chenxu14

OK, fixed

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@dutor dutor added ready-for-testing PR: ready for the CI test and removed ready-for-testing PR: ready for the CI test labels Jul 27, 2020
@dangleptr dangleptr merged commit 5ed456b into vesoft-inc:master Jul 27, 2020
xuguruogu pushed a commit to xuguruogu/nebula that referenced this pull request Aug 5, 2020
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…soft-inc#2243)

Co-authored-by: kyle.cao <kyle.cao@vesoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants