On 2025-Mar-26, Suraj Kharage wrote: > Yes, I agree. As Peter said, it is now inline with other commands. > > I have reviewed the patch and it looks good to me.
Thanks for reviewing! > Since we are removing the SET keyword, how about removing that from the > below comment as well. > > /* > * Propagate the change to children. For SET NO INHERIT, we don't > * recursively affect children, just the immediate level. > */ > > This is the comment from ATExecAlterConstrInheritability(). Ah right. Fixed that and pushed. -- Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/