> 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/




Attachment: v11-0001-Prevent-inherited-CHECK-constraints-from-being-w.patch
Description: Binary data

Attachment: v11-0002-doc-Clarify-inherited-constraint-behavior.patch
Description: Binary data

Attachment: v11-0003-doc-Clarify-ALTER-CONSTRAINT-enforceability-beha.patch
Description: Binary data

Reply via email to