Re: BUG #18344: Pruning tables partitioned by bool range fails with invalid strategy
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: exclusion@gmail.com
Cc: pgsql-bugs@lists.postgresql.org, David Rowley <dgrowleyml@gmail.com>
Date: 2024-02-15T16:28:23Z
Lists: pgsql-bugs
PG Bug reporting form <noreply@postgresql.org> writes: > The following query: > CREATE TABLE t (b bool, i int) PARTITION BY RANGE (b, i); > CREATE TABLE tp PARTITION OF t FOR VALUES FROM (false, 0) TO (false, 1); > SELECT * FROM t WHERE b IS NOT true; > fails with ERROR: invalid strategy number 0. > Reproduced on REL_12_STABLE .. master. > The first bad commit for this anomaly is e0693faf7. What seems to be happening is that gen_prune_step_op is getting op_is_ne = true and doing this: /* * For clauses that contain an <> operator, set opstrategy to * InvalidStrategy to signal get_matching_list_bounds to do the right * thing. */ opstep->opstrategy = op_is_ne ? InvalidStrategy : opstrategy; but then we're failing in get_matching_range_bounds, ie somebody taught get_matching_list_bounds to do the right thing but not any of the other code paths. I'm also wondering how we got there in the first place. It looks like match_boolean_partition_clause thinks it can translate "b IS NOT true" to "b <> true", which is flat wrong --- it gives the wrong result for null. regards, tom lane
Commits
-
Minor corrections for partition pruning
- d2ca9a50b5b9 17.0 landed
-
Fix incorrect pruning of NULL partition for boolean IS NOT clauses
- 3ffcd24c29c6 12.19 landed
- 3850fcca69b5 13.15 landed
- f9c8f7ccd6e3 14.12 landed
- 1b3495e29db6 15.7 landed
- fb95cc72bfe7 16.3 landed
- 4c2369ac5d0a 17.0 landed