This goes through UserIdentityCache, so we are guaranteed that multiple fetches for the same user ID will result in at most a single identity call.
Depends on D13389
Paths
| Differential D13390 Authored by ashoat on Sep 19 2024, 3:44 PM.
Details Summary This goes through UserIdentityCache, so we are guaranteed that multiple fetches for the same user ID will result in at most a single identity call. Depends on D13389 Test Plan For now, just Flow. I'll test this at a later point in the stack; for now, putting up diffs following the team's recent pattern to get feedback
Diff Detail
Event TimelineHarbormaster completed remote builds in B31718: Diff 44348.Sep 19 2024, 4:02 PM2024-09-19 16:02:19 (UTC-7) This revision is now accepted and ready to land.Sep 20 2024, 1:31 AM2024-09-20 01:31:43 (UTC-7) Closed by commit rCOMM233ae45c5c70: [lib] Introduce findUserIdentities to ProcessDMOperationUtilities (authored by ashoat). · Explain WhySep 20 2024, 6:16 AM2024-09-20 06:16:11 (UTC-7) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 44368 lib/shared/dm-ops/dm-op-spec.js
lib/shared/dm-ops/process-dm-ops.js
|