forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit92c651f
committed
Fix an oversight in mergejoin planning: the planner would reject a
mergejoin possibility where the inner rel was less well sorted thanthe outer (ie, it matches some but not all of the merge clauses thatcan work with the outer), if the inner path in question is also theoverall cheapest path for its rel. This is an old bug, but I'm notsure it's worth back-patching, because it's such a corner case.Noted while investigating a test case from Peter Hardman.1 parent144b0ae commit92c651f
1 file changed
+10
-5
lines changedLines changed: 10 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
542 | 542 |
| |
543 | 543 |
| |
544 | 544 |
| |
545 |
| - | |
546 |
| - | |
| 545 | + | |
| 546 | + | |
| 547 | + | |
| 548 | + | |
| 549 | + | |
547 | 550 |
| |
548 | 551 |
| |
549 | 552 |
| |
| |||
568 | 571 |
| |
569 | 572 |
| |
570 | 573 |
| |
571 |
| - | |
| 574 | + | |
| 575 | + | |
572 | 576 |
| |
573 | 577 |
| |
574 | 578 |
| |
| |||
603 | 607 |
| |
604 | 608 |
| |
605 | 609 |
| |
606 |
| - | |
| 610 | + | |
| 611 | + | |
607 | 612 |
| |
608 | 613 |
| |
609 | 614 |
| |
|
0 commit comments
Comments
(0)