Perhaps we can merge some of the code cleanup that it contained, such as using XID_FMT everywhere and creating a type for the kind of page returned by TransactionIdToPage() to make the code cleaner.
Agree, I think this is a good idea.
Is your patch functionally the same as the PostgresPro implementation?
Yes, it is. It basically is PostgresPro implementation, not a concept or smth.