forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit5e7fa18

Etsuro Fujita
postgres_fdw: Fix assertion in estimate_path_cost_size().
Commit08d2d58 added an assertion assuming that the retrieved_rowsestimate for a foreign relation, which is re-used to cost pre-sortedforeign paths with local stats, is set to at least one row inestimate_path_cost_size(), which isn't correct because if the relationis a foreign table with tuples=0, the estimate would be set to 0 therewhen not using remote estimates.Per bug #16807 from Alexander Lakhin. Back-patch to v13 where theaforementioned commit went in.Author: Etsuro FujitaReviewed-by: Kyotaro HoriguchiDiscussion:https://postgr.es/m/16807-9fe4e08fbaa5c7ce%40postgresql.org1 parent0ff865f commit5e7fa18
File tree
3 files changed
+31
-1
lines changed- contrib/postgres_fdw
- expected
- sql
3 files changed
+31
-1
lines changedLines changed: 18 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
613 | 613 |
| |
614 | 614 |
| |
615 | 615 |
| |
| 616 | + | |
| 617 | + | |
| 618 | + | |
| 619 | + | |
| 620 | + | |
| 621 | + | |
| 622 | + | |
| 623 | + | |
| 624 | + | |
| 625 | + | |
| 626 | + | |
| 627 | + | |
| 628 | + | |
| 629 | + | |
| 630 | + | |
| 631 | + | |
| 632 | + | |
| 633 | + | |
616 | 634 |
| |
617 | 635 |
| |
618 | 636 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2923 | 2923 |
| |
2924 | 2924 |
| |
2925 | 2925 |
| |
2926 |
| - | |
| 2926 | + | |
2927 | 2927 |
| |
2928 | 2928 |
| |
2929 | 2929 |
| |
|
Lines changed: 12 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
309 | 309 |
| |
310 | 310 |
| |
311 | 311 |
| |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
312 | 324 |
| |
313 | 325 |
| |
314 | 326 |
| |
|
0 commit comments
Comments
(0)