Code review of get_qual_for_list.

We need not consider the case where both nulltest1 and nulltest2 are
NULL; the partition either accepts nulls or it does not.

Jeevan Ladhe.  I added an assertion.
pull/20/merge
Robert Haas 9 years ago
parent 9ae2661fe1
commit 85c2b9a15a
  1. 5
      src/backend/catalog/partition.c

@ -1383,15 +1383,14 @@ get_qual_for_list(PartitionKey key, PartitionBoundSpec *spec)
if (nulltest1)
result = list_make2(nulltest1, opexpr);
else if (nulltest2)
else
{
Expr *or;
Assert(nulltest2 != NULL);
or = makeBoolExpr(OR_EXPR, list_make2(nulltest2, opexpr), -1);
result = list_make1(or);
}
else
result = list_make1(opexpr);
return result;
}

Loading…
Cancel
Save