Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit e4a7c03

Browse files
committed
Address ccvalid/ccnoinherit in TupleDesc support functions.
equalTupleDescs() neglected both of these ConstrCheck fields, and CreateTupleDescCopyConstr() neglected ccnoinherit. At this time, the only known behavior defect resulting from these omissions is constraint exclusion disregarding a CHECK constraint validated by an ALTER TABLE VALIDATE CONSTRAINT statement issued earlier in the same transaction. Back-patch to 9.2, where these fields were introduced.
1 parent fb91874 commit e4a7c03

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/backend/access/common/tupdesc.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,6 +204,7 @@ CreateTupleDescCopyConstr(TupleDesc tupdesc)
204204
if (constr->check[i].ccbin)
205205
cpy->check[i].ccbin = pstrdup(constr->check[i].ccbin);
206206
cpy->check[i].ccvalid = constr->check[i].ccvalid;
207+
cpy->check[i].ccnoinherit = constr->check[i].ccnoinherit;
207208
}
208209
}
209210

@@ -417,7 +418,9 @@ equalTupleDescs(TupleDesc tupdesc1, TupleDesc tupdesc2)
417418
for (j = 0; j < n; check2++, j++)
418419
{
419420
if (strcmp(check1->ccname, check2->ccname) == 0 &&
420-
strcmp(check1->ccbin, check2->ccbin) == 0)
421+
strcmp(check1->ccbin, check2->ccbin) == 0 &&
422+
check1->ccvalid == check2->ccvalid &&
423+
check1->ccnoinherit == check2->ccnoinherit)
421424
break;
422425
}
423426
if (j >= n)

0 commit comments

Comments
 (0)