[SERVER-39936] Use PeriodicRunner handles to simplify shutdown ordering Created: 01/Mar/19 Updated: 29/Oct/23 Resolved: 21/Jun/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 4.2.0-rc5, 4.0.12, 4.3.1 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Benjamin Caimano (Inactive) | Assignee: | Benjamin Caimano (Inactive) |
| 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 | ||||||||||||||||||||||||
| Backport Requested: |
v4.2, v4.0
|
||||||||||||||||||||||||
| Sprint: | Service Arch 2019-03-11, Service Arch 2019-03-25, Service Arch 2019-04-08, Service Arch 2019-04-22, Service Arch 2019-05-06, Service Arch 2019-05-20, Service Arch 2019-06-03, Service Arch 2019-06-17, Service Arch 2019-07-01 | ||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Linked BF Score: | 54 | ||||||||||||||||||||||||
| Description |
|
It doesn't. |
| Comments |
| Comment by Githook User [ 30/Jul/19 ] |
|
Author: {'name': 'Ben Caimano', 'email': 'ben.caimano@10gen.com'}Message: (cherry picked from commit bf4a3cff4dc5572f2e97cb5279fe63c8227187e0)
(cherry picked from commit 1eff33bd1a8d48eb607675f87faf1836ba325006) This is a partial backport. Parts of the original commits listed do not |
| Comment by Githook User [ 25/Jul/19 ] |
|
Author: {'name': 'Ben Caimano', 'email': 'ben.caimano@10gen.com'}Message: |
| Comment by Githook User [ 21/Jun/19 ] |
|
Author: {'name': 'Ben Caimano', 'email': 'ben.caimano@10gen.com'}Message: |
| Comment by Benjamin Caimano (Inactive) [ 19/Mar/19 ] |
|
Yeah, it's related. I'm trying to have a more cohesive approach for who shuts down with handles, there's some oddness where the PeriodicRunner is responsible for shutting down other pieces by dying earlier than one would expect. |
| Comment by Matthew Saltz (Inactive) [ 19/Mar/19 ] |
|
Is this related to |
| Comment by Benjamin Caimano (Inactive) [ 09/Mar/19 ] |
|
Going in a different direction on this one now that I have some understanding of the intended contract for PeriodicRunner jobs. (The metaphor is crond.) |