- Notifications
You must be signed in to change notification settings - Fork5
Commit3d660d3
committed
Fix assorted oversights in range selectivity estimation.
calc_rangesel() failed outright when comparing range variables to emptyconstant ranges with < or >=, as a result of missing cases in a switch.It also produced a bogus estimate for > comparison to an empty range.On top of that, the >= and > cases were mislabeled throughout. Fornonempty constant ranges, they managed to produce the right answersanyway as a result of counterbalancing typos.Also, default_range_selectivity() omitted cases for elem <@ range,range &< range, and range &> range, so that rather dubious defaultswere applied for these operators.In passing, rearrange the code in rangesel() so that the elem <@ rangecase is handled in a less opaque fashion.Report and patch by Emre Hasegeli, some additional work by me1 parent68fa75f commit3d660d3
File tree
4 files changed
+72
-18
lines changed- src
- backend/utils/adt
- include/catalog
- test/regress
- expected
- sql
4 files changed
+72
-18
lines changedLines changed: 34 additions & 16 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
73 | 73 |
| |
74 | 74 |
| |
75 | 75 |
| |
| 76 | + | |
76 | 77 |
| |
77 | 78 |
| |
78 | 79 |
| |
| |||
86 | 87 |
| |
87 | 88 |
| |
88 | 89 |
| |
| 90 | + | |
| 91 | + | |
89 | 92 |
| |
90 | 93 |
| |
91 | 94 |
| |
| |||
109 | 112 |
| |
110 | 113 |
| |
111 | 114 |
| |
112 |
| - | |
| 115 | + | |
113 | 116 |
| |
114 | 117 |
| |
115 | 118 |
| |
| |||
186 | 189 |
| |
187 | 190 |
| |
188 | 191 |
| |
189 |
| - | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
190 | 201 |
| |
191 |
| - | |
| 202 | + | |
| 203 | + | |
192 | 204 |
| |
193 |
| - | |
194 |
| - | |
| 205 | + | |
195 | 206 |
| |
196 | 207 |
| |
197 | 208 |
| |
198 | 209 |
| |
199 | 210 |
| |
200 |
| - | |
| 211 | + | |
| 212 | + | |
201 | 213 |
| |
202 | 214 |
| |
203 | 215 |
| |
| |||
270 | 282 |
| |
271 | 283 |
| |
272 | 284 |
| |
| 285 | + | |
273 | 286 |
| |
274 | 287 |
| |
275 | 288 |
| |
276 | 289 |
| |
277 | 290 |
| |
278 |
| - | |
| 291 | + | |
| 292 | + | |
279 | 293 |
| |
280 | 294 |
| |
281 | 295 |
| |
| 296 | + | |
282 | 297 |
| |
| 298 | + | |
283 | 299 |
| |
284 |
| - | |
285 |
| - | |
286 |
| - | |
287 |
| - | |
288 |
| - | |
289 |
| - | |
290 | 300 |
| |
291 | 301 |
| |
292 | 302 |
| |
293 |
| - | |
294 | 303 |
| |
| 304 | + | |
| 305 | + | |
| 306 | + | |
295 | 307 |
| |
296 | 308 |
| |
297 | 309 |
| |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
298 | 316 |
| |
299 | 317 |
| |
300 | 318 |
| |
| |||
443 | 461 |
| |
444 | 462 |
| |
445 | 463 |
| |
446 |
| - | |
| 464 | + | |
447 | 465 |
| |
448 | 466 |
| |
449 | 467 |
| |
450 | 468 |
| |
451 | 469 |
| |
452 |
| - | |
| 470 | + | |
453 | 471 |
| |
454 | 472 |
| |
455 | 473 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1723 | 1723 |
| |
1724 | 1724 |
| |
1725 | 1725 |
| |
1726 |
| - | |
| 1726 | + | |
1727 | 1727 |
| |
1728 | 1728 |
| |
1729 |
| - | |
| 1729 | + | |
1730 | 1730 |
| |
1731 | 1731 |
| |
1732 | 1732 |
| |
|
Lines changed: 32 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
260 | 260 |
| |
261 | 261 |
| |
262 | 262 |
| |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
263 | 268 |
| |
264 | 269 |
| |
265 | 270 |
| |
| |||
287 | 292 |
| |
288 | 293 |
| |
289 | 294 |
| |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
290 | 322 |
| |
291 | 323 |
| |
292 | 324 |
| |
|
Lines changed: 4 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
67 | 67 |
| |
68 | 68 |
| |
69 | 69 |
| |
| 70 | + | |
70 | 71 |
| |
71 | 72 |
| |
72 | 73 |
| |
| 74 | + | |
| 75 | + | |
| 76 | + | |
73 | 77 |
| |
74 | 78 |
| |
75 | 79 |
| |
|
0 commit comments
Comments
(0)