[SERVER-60133] TenantMigrationDonorService has multiple sources of truth for the current state Created: 21/Sep/21  Updated: 17/Oct/23

Status: Backlog
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Major - P3
Reporter: Matt Broadstone Assignee: Backlog - Service Architecture
Resolution: Unresolved Votes: 0
Labels: FY22Q4-cand
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Service Arch
Participants:

 Description   

The TenantMigrationDonorService sometimes uses the _stateDoc as a source of truth for the current state, and other times uses _durableState.state. Additionally, we sometimes save a copy of the in-memory state document off in order to update the durable state document which has lead to data races (SERVER-61565). We should reconsider how we track state to eliminate this class of error, and to make the code more readable for future maintainers.



 Comments   
Comment by Esha Maharishi (Inactive) [ 18/Nov/21 ]

Re-opening after seeing SERVER-61565.

Comment by Steven Vannelli [ 08/Nov/21 ]

Closing as "won't do" as this is not currently a high priority.

Generated at Thu Feb 08 05:49:02 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.