@ameade Yeah, like I said I don't think there's really a problem. Just wanted to make sure someone else specifically looked at it just to double-check.

Also for the deserialize_schedule_metadata I basically was copying the pattern set by the serialize / deserialize datetime stuff which also operates in-place. But I have no strong attachment to that approach. I do think if we change one we should change the other as well. Like you said tho, separate issue.