[15:47:24] AaronSchulz: agreed. It can get tricky for cases where the array is mutable via subclasses or hooks, but for any wholly internal ones, a value class would likely be fine and improve confidence. [15:47:41] AaronSchulz: is https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ConfirmEdit/+/674742 still relevant? going through a few old patches open in the team. ref T336004 [15:47:43] T336004: Switch ConfirmEdit extension from MainStash DB to mcrouter-primary-dc - https://phabricator.wikimedia.org/T336004 [17:28:50] Krinkle: I'll close that and spin off the WRITE_BACKGROUND. We agreed to add a new MW services method for this instead.