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
Description: Binary data
