> On Jun 17, 2026, at 11:27, jian he <[email protected]> wrote: > > On Tue, Jun 9, 2026 at 8:32 AM Chao Li <[email protected]> wrote: >> >> In v10, I split the “because” part to a errdetail, also moved out NOT >> ENFORCED out of the translate message. >> > > ATCheckCheckConstrHasEnforcedParent > `````` > if (constraints_equivalent(parenttup, contuple, > RelationGetDescr(conrel))) > { > `````` > The above IF condition is basically always true (see > MergeConstraintsIntoExisting), unless an inherited check constraint > has a > different definition, which should not happen. > So I did a quick refactor here, which also drops the nesting level down by > one. > > Other than that, v10 looks good to me. > > > > -- > jian > https://www.enterprisedb.com/ > <v10-0001-misc-refactor-ATCheckCheckConstrHasEnforcedParent.nocfbot>
Thanks for your suggestion, I think that is correct. PFA v11 - 0001 integrated Jian’s change; 0002 and 0003 unchanged. Best regards, -- Chao Li (Evan) HighGo Software Co., Ltd. https://www.highgo.com/
v11-0001-Prevent-inherited-CHECK-constraints-from-being-w.patch
Description: Binary data
v11-0002-doc-Clarify-inherited-constraint-behavior.patch
Description: Binary data
v11-0003-doc-Clarify-ALTER-CONSTRAINT-enforceability-beha.patch
Description: Binary data
