-
Type: Task
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
None
-
Fully Compatible
-
Server Serverless 2022-12-12, Server Serverless 2022-12-26
Change the tenant_migration_access_blocker_util and the tenant_migration_access_blocker_registry to use a vector<tenant_id> instead of the vector<string> and all other places where we use StringData instead of a TenantId object. The unit tests should also be changed. In case of this ticket done asynchronously we should have support for both TenantId object and StringData until the other tickets are done. We can either mark the change blocked + depend on; or we can create a TODO ticket to delete the old api still using StringData over TenantId.
We should also identify and create a TODO ticket (SERVER-71669) as part of this work to identify any place that will be left still using a StringData for backward compatibility with MTM and Shard Merge (since both protocol are still using a StringData type).
We cannot fully modify the TenantMigrationAccessBlockerRegistry until all migration protocol are moved to use TenantId, therefor SERVER-71673 was created to handle the refactoring to only use TenantId.
- related to
-
SERVER-70913 Investigate changes needed to ShardSplit code in order to use TenantId/DatabaseName objects
- Closed
-
SERVER-71669 Remove TODO to no longer use StringData when manipulating TenantIds
- Closed
-
SERVER-71673 Refactor TenantMigrationAccessBlockerRegistry internal data structure to use TenantId
- Closed