On Mon, 2026-08-03 at 11:07 +0530, Amit Kapila wrote:
> OTOH, I am also fine if you prefer to remove the retain_dead_tuples
> check entirely from the ENABLE path and keep it in other existing
> paths as in attached. Actually, that will slightly simplify the code
> as well.
The patch looks good to me.
For approach 1 versus 2, I'll defer to you and authors/reviewers of the
feature who understand the use cases better. I slightly prefer the
above approach (don't connect at ENABLE time) because side effects at
DDL time are hard to reason about.
Regards,
Jeff Davis
