[SERVER-71849] Named Pipes: Extend reader retry loop from 1 sec to 30 sec Created: 05/Dec/22 Updated: 29/Oct/23 Resolved: 06/Dec/22 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.3.0-rc0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Kevin Cherkauer | Assignee: | Kevin Cherkauer |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Operating System: | ALL | ||||||||||||
| Participants: | |||||||||||||
| Linked BF Score: | 20 | ||||||||||||
| Description |
|
Second fix attempt for BF-26979. Extending the named pipe reader retry loop to 30 seconds, using appropriate backoff on how fast they retry after the first one second, so they don't spin like crazy on a system that is already overloaded. (Retry is currently done every 1 msec for up to 1,000 times, then it gives up. We want these to be dense during the first second as in production the waits add to the query latency, and the pipe is in the process of being created so should show up very quickly.) The retry loops are in the NamedPipeInput::doOpen() methods of named_pipe_posix.cpp and named_pipe_posix.cpp. The InputStream() constructor in input_stream.h also needs to add more detail to the error message here:
FYI arun.banala@mongodb.com yoonsoo.kim@mongodb.com |
| Comments |
| Comment by Githook User [ 06/Dec/22 ] |
|
Author: {'name': 'Kevin Cherkauer', 'email': 'kevin.cherkauer@mongodb.com', 'username': 'kevin-cherkauer'}Message: |