Skip to content

[feat] make min_reserved_fds runtime-configurable - #447

Merged
qiongtubao merged 1 commit into
mainfrom
feature/min_reserved_fds
Sep 7, 2026
Merged

[feat] make min_reserved_fds runtime-configurable#447
qiongtubao merged 1 commit into
mainfrom
feature/min_reserved_fds

Conversation

@qiongtubao

Copy link
Copy Markdown
  • add 'min-reserved-fds' config option (default 32, MODIFIABLE)
  • replace hardcoded CONFIG_MIN_RESERVED_FDS with server.min_reserved_fds in event loop sizing, OOM score adj and adjustOpenFilesLimit
  • add tryResizeSetAeSize helper to resize aeSet on value change
  • update swapBatchTest to size aeSet consistently

@qiongtubao
qiongtubao force-pushed the feature/min_reserved_fds branch 2 times, most recently from 972b48d to 93a5275 Compare September 7, 2026 03:36
- add 'min-reserved-fds' config option (default 32, IMMUTAB)
- replace hardcoded CONFIG_MIN_RESERVED_FDS with server.min_reserved_fds
  in event loop sizing, OOM score adj and adjustOpenFilesLimit
- add tryResizeSetAeSize helper to resize aeSet on value change
- update swapBatchTest to size aeSet consistently
@qiongtubao
qiongtubao force-pushed the feature/min_reserved_fds branch from 93a5275 to eb37d4a Compare September 7, 2026 04:32
@evazca

evazca commented Sep 7, 2026

Copy link
Copy Markdown

lgtm

@qiongtubao
qiongtubao merged commit a81df11 into main Sep 7, 2026
9 of 10 checks passed
@qiongtubao
qiongtubao deleted the feature/min_reserved_fds branch September 7, 2026 06:27
qiongtubao added a commit that referenced this pull request Sep 7, 2026
- add 'min-reserved-fds' config option (default 32, IMMUTAB)
- replace hardcoded CONFIG_MIN_RESERVED_FDS with server.min_reserved_fds
  in event loop sizing, OOM score adj and adjustOpenFilesLimit
- add tryResizeSetAeSize helper to resize aeSet on value change
- update swapBatchTest to size aeSet consistently

Co-authored-by: gd.zhou <gd.zhou@trip.com>
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.

2 participants