On Wed, 2024-03-06 at 12:01 +0000, Li, Yong wrote: > Rebase the patch against the latest HEAD.
The upgrade logic could use more comments explaining what's going on and why. As I understand it, it's a one-time conversion that needs to happen between 16 and 17. Is that right? Was the way CLOG is upgraded already decided in some earlier discussion? Given that the CLOG is append-only and gets truncated occasionally, I wonder whether we can just have some marker that xids before some number are the old CLOG, and xids beyond that number are in the new CLOG. I'm not necessarily suggesting that; just an idea. Regards, Jeff Davis