aboutsummaryrefslogtreecommitdiff
path: root/src/backend/access/gist/gistxlog.c
diff options
context:
space:
mode:
authorDavid Rowley <drowley@postgresql.org>2019-07-12 19:12:38 +1200
committerDavid Rowley <drowley@postgresql.org>2019-07-12 19:12:38 +1200
commitcfde23493938b543d281c509527f0f5b16fca9ff (patch)
treed627999c9158c1cd3c3e534ac7de5e712d793669 /src/backend/access/gist/gistxlog.c
parent0cea6eb5a5f2948c411706cabfde32ce61df0d7a (diff)
downloadpostgresql-cfde23493938b543d281c509527f0f5b16fca9ff.tar.gz
postgresql-cfde23493938b543d281c509527f0f5b16fca9ff.zip
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
Diffstat (limited to 'src/backend/access/gist/gistxlog.c')
0 files changed, 0 insertions, 0 deletions