forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit1b68010
committed
Change bms_add_range to be a no-op for empty ranges
In commit8494064, bms_add_range was added with an API to fail withan error if an empty range was specified. This seems arbitrary andunhelpful, so turn that case into a no-op instead. Callers that requirefurther verification on the arguments or result can apply them bythemselves.This fixes the bug that partition pruning throws an API error for a caseinvolving the default partition of a default partition, as in theincluded test case.Reported-by: Rajkumar Raghuwanshi <rajkumar.raghuwanshi@enterprisedb.com>Diagnosed-by: Tom Lane <tgl@sss.pgh.pa.us>Discussion:https://postgr.es/m/16590.1532622503@sss.pgh.pa.us1 parent7dc5a96 commit1b68010
File tree
3 files changed
+27
-2
lines changed- src
- backend/nodes
- test/regress
- expected
- sql
3 files changed
+27
-2
lines changedLines changed: 5 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
867 | 867 |
| |
868 | 868 |
| |
869 | 869 |
| |
| 870 | + | |
| 871 | + | |
| 872 | + | |
| 873 | + | |
870 | 874 |
| |
871 | 875 |
| |
872 | 876 |
| |
| |||
878 | 882 |
| |
879 | 883 |
| |
880 | 884 |
| |
881 |
| - | |
882 |
| - | |
883 | 885 |
| |
884 | 886 |
| |
885 | 887 |
| |
886 | 888 |
| |
887 | 889 |
| |
| 890 | + | |
888 | 891 |
| |
889 | 892 |
| |
890 | 893 |
| |
|
Lines changed: 15 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1180 | 1180 |
| |
1181 | 1181 |
| |
1182 | 1182 |
| |
| 1183 | + | |
| 1184 | + | |
| 1185 | + | |
| 1186 | + | |
| 1187 | + | |
| 1188 | + | |
| 1189 | + | |
| 1190 | + | |
| 1191 | + | |
| 1192 | + | |
| 1193 | + | |
| 1194 | + | |
| 1195 | + | |
| 1196 | + | |
| 1197 | + | |
1183 | 1198 |
| |
1184 | 1199 |
| |
1185 | 1200 |
| |
|
Lines changed: 7 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
173 | 173 |
| |
174 | 174 |
| |
175 | 175 |
| |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
176 | 183 |
| |
177 | 184 |
| |
178 | 185 |
| |
|
0 commit comments
Comments
(0)