RecordId contains an explicit Null state when created without any data. There are currently usages of RecordId in a boost::optional which means we need to double check the nullability of the given element.

This ticket is about reconsidering usages of this in favour of a plain RecordId with the Null state to avoid unnecessary wrapping of RecordId into a boost::optional.