[SERVER-39570] Separate NITP::consumeTasks into an entry and an impl Created: 13/Feb/19 Updated: 08/Jan/24 Resolved: 29/May/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Networking |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | Benjamin Caimano (Inactive) | Assignee: | Backlog - Service Architecture |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Service Arch
|
||||||||
| Participants: | |||||||||
| Description |
|
So we currently have a somewhat difficult to parse if block in the NetworkInterfaceThreadPool. I think we can probably simplify this into consumeTasks() for actually doing the work and scheduleCosumeTasks() for registering our work onto the executor. There's also some weirdness around the booleans _registeredAlarm and _consumingTasks that probably could benefit from becoming an enum. |
| Comments |
| Comment by Benjamin Caimano (Inactive) [ 29/May/19 ] |
|
Looks like this was taken care of after all |
| Comment by Benjamin Caimano (Inactive) [ 13/Feb/19 ] |
|
mira.carey@mongodb.com, this is definitely one of those backloggable tech-debt neweng tickets. |