It would be fine to do it that way right now. I was mostly worried about somehow saving to the database in the middle of that operation, which shouldn't be possible as long as we're synchronous and single-threaded. I think the copy isn't a big deal since we're dealing with small, well-defined structures (it will always be a dict of lists of strings). I don't think we would hit any edge cases with copy here.