-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
add some config #4310
add some config #4310
Conversation
conf/nebula-graphd.conf.default
Outdated
# Network IO threads number, 0 means physical cpu core num | ||
--num_netio_threads=0 | ||
# Worker threads number to handle request, 0 means physical cpu core num | ||
--num_worker_threads=0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have these two config on line 62-65
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM on storage part. What was the reason for snapshot_part_rate_limit
, snapshot_batch_size
... appearing only in production config not default config before?
conf/nebula-storaged.conf.default
Outdated
--num_io_threads=16 | ||
# Worker threads number to handle request | ||
--num_worker_threads=32 | ||
# Maximum subtasks to run concurrently |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
be more precise "... for admin jobs"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌
Will we change it in Open Source edition, or just in Enterprise edition? cc @xiajingchun |
What type of PR is this?
What problem(s) does this PR solve?
Issue(s) number:
close #4246
Description:
Add some configs into configuration files which are commonly used.
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:
Checklist:
Tests:
Affects:
Note sure if all these newly added configs are in doc.
Release notes:
Please confirm whether to be reflected in release notes and how to describe: