[SERVER-20030] ForwardingCatalogManager::shutdown races with _replaceCatalogManager Created: 19/Aug/15 Updated: 25/Jan/17 Resolved: 29/Jan/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 3.2.3, 3.3.2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Andy Schwerin | Assignee: | Spencer Brody (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | code-and-test, csrsupgrade | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Operating System: | ALL | ||||||||||||
| Backport Completed: | |||||||||||||
| Sprint: | Sharding F (01/29/16) | ||||||||||||
| Participants: | |||||||||||||
| Description |
|
If _replaceCatalogManager gets scheduled in a task executor, but does not start to run before another thread executes ForwardingCatalogManager::shutdown(), the shutdown might complete before the replace runs, which could lead to the FCM being destroyed while there is still an outstanding operation on it. |
| Comments |
| Comment by Githook User [ 29/Jan/16 ] |
|
Author: {u'username': u'stbrody', u'name': u'Spencer T Brody', u'email': u'spencer@mongodb.com'}Message: (cherry picked from commit 06bd28f354ec36227cd085660fee35c3f8a8543d) |
| Comment by Githook User [ 29/Jan/16 ] |
|
Author: {u'username': u'stbrody', u'name': u'Spencer T Brody', u'email': u'spencer@mongodb.com'}Message: |
| Comment by Andy Schwerin [ 31/Aug/15 ] |
|
Yes, spencer, we do. |
| Comment by Spencer Brody (Inactive) [ 27/Aug/15 ] |
|
Don't we shut down the ForwardingCatalogManager here? |