-
Notifications
You must be signed in to change notification settings - Fork 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
Configure Node GC Threshold #362
Conversation
@@ -173,6 +173,8 @@ configured on client nodes. | |||
sub-schedulers this server will handle. This can be used to restrict the | |||
evaluations that worker threads will dequeue for processing. This | |||
defaults to all available schedulers. | |||
* `node_gc_threshold` This is a string with a unit suffix, such as "300ms", | |||
"1.5h" or "25m". Valid time units are "ns", "us" (or "µs"), "ms", "s", "m", "h". Controls how "old" a node must be to be collected by GC |
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.
Would you mind formatting this to 80 characters. I would also reword the last sentence to something like: "Controls how long a node must be in a terminal state before it is garbage collected and purged from the system."
Small comment but looks great! |
Second time I forget about 80 characters will and rule to my linter. |
Awesome! Thanks so much! |
…shicorp#362) * issue358 Snapshot can attempt to delete a negative range from log * issue 358 change it to strict inequality
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
Close #333