-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix: Keep working when EMFILE is reached #2158
Conversation
Done by upgrading to warp 3.3.19
Do you think we can craft an io test that runs into the EMFILE problem without the patch? |
We'd be testing Warp if we do that, so I think we shouldn't. However, basically we need to dynamically change the file descriptor limit(can be done with prlimit) to something like 1, make requests and see postgrest fail. |
Doing a
(Ideally the EMFILE error would bubble up to the client instead of waiting but at least we keep running) |
The above looks like a recovery test, so I've listed it on #1766 to add it later if it's worth it. |
Agree with both! |
Done by upgrading to warp 3.3.19
Done by upgrading to warp 3.3.19
Done by upgrading to warp 3.3.19
Done by upgrading to warp 3.3.19
Done by upgrading to warp 3.3.19
Closes #2042.
TODO