Differential D20224
Remove "Effective User" attachment from Repository Identities Authored by epriestley on Feb 28 2019, 4:30 PM. Tags None Referenced Files
Subscribers None
Details It's possible for an Idenitity to be bound to user X, then for that user to be deleted, e.g. with bin/remove destroy X. In this case, we'll fail to load/attach the user to the identity. Currently, we don't actually use this anywhere, so just stop loading it. Once we start using it (if we ever do), we could figure out what an appropriate policy is in this case. Browsed around Diffusion, grepped for affected "effective user" methods and found no callsites.
Diff Detail
Event TimelineComment Actions (Even if we do need this eventually, we clearly don't need it terribly often, so putting it behind a needEffectiveUsers(true) check would probably make sense.) Comment Actions
This turns out to be a bit tricky and doesn't really do anything for us today, so I'll just note it on T12164. This is tricky because destroying a user may possibly require us to update an arbitrary number of RepositoryIdentity objects, and the most correct update is to rebuild the identity completely (i.e., recompute the "guess"). I think we'd have to queue some stuff into the task queue to make this scalable in the general case, so a fully general version of this requires a fair bit of code shuffling. |