[SERVER-46507] Convert member variable from migrationId in MigrationDestinationManager to boost::optional Created: 28/Feb/20 Updated: 29/Oct/23 Resolved: 09/Mar/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.4.0-rc0, 4.7.0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Alexander Taskov (Inactive) | Assignee: | Alexander Taskov (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 | ||||
| Backport Requested: |
v4.4
|
||||
| Sprint: | Sharding 2020-03-09, Sharding 2020-03-23 | ||||
| Participants: | |||||
| Description |
|
Currently, the migrationId member variable is stored as a normal member variable of the MigrationDestinationManager class. However, it's undefined at construction time as it gets initialized with the state passed in from the recvChunkStart command. So it should be converted to a boost::optional so that it doesn't have to make the MigrationDestinationManager a friend. |
| Comments |
| Comment by Githook User [ 25/Mar/20 ] |
|
Author: {'email': 'alex.taskov@mongodb.com', 'name': 'Alex Taskov', 'username': 'alextaskov'}Message: (cherry picked from commit bb32cfb51f36b3d088c5e1f78cebb5b9ab29d498) |
| Comment by Githook User [ 09/Mar/20 ] |
|
Author: {'name': 'Alex Taskov', 'username': 'alextaskov', 'email': 'alex.taskov@mongodb.com'}Message: |