]> granicus.if.org Git - postgresql/commit
Fix RANGE partition pruning with multiple boolean partition keys
authorDavid Rowley <drowley@postgresql.org>
Fri, 12 Jul 2019 07:10:12 +0000 (19:10 +1200)
committerDavid Rowley <drowley@postgresql.org>
Fri, 12 Jul 2019 07:10:12 +0000 (19:10 +1200)
commit757f1baa8c13ffe7abd7b0f49f787a1c007cdf89
tree38c8821b2fed254b1a5953bb86c358f1957bf032
parent5918b82614513e1262c6c3ed286e4e1212a01bc3
Fix RANGE partition pruning with multiple boolean partition keys

match_clause_to_partition_key incorrectly would return
PARTCLAUSE_UNSUPPORTED if a bool qual could not be matched to the current
partition key.  This was a problem, as it causes the calling function to
discard the qual and not try to match it to any other partition key.  If
there was another partition key which did match this qual, then the qual
would not be checked again and we could fail to prune some partitions.

The worst this could do was to cause partitions not to be pruned when they
could have been, so there was no danger of incorrect query results here.

Fix this by changing match_boolean_partition_clause to have it return a
PartClauseMatchStatus rather than a boolean value.  This allows it to
communicate if the qual is unsupported or if it just does not match this
particular partition key, previously these two cases were treated the
same.  Now, if match_clause_to_partition_key is unable to match the qual
to any other qual type then we can simply return the value from the
match_boolean_partition_clause call so that the calling function properly
treats the qual as either unmatched or unsupported.

Reported-by: Rares Salcudean
Reviewed-by: Amit Langote
Backpatch-through: 11 where partition pruning was introduced
Discussion: https://postgr.es/m/CAHp_FN2xwEznH6oyS0hNTuUUZKp5PvegcVv=Co6nBXJ+mC7Y5w@mail.gmail.com
src/backend/partitioning/partprune.c
src/test/regress/expected/partition_prune.out
src/test/regress/sql/partition_prune.sql