forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit18f5108
committed
Fix oversight in recent MULTIEXPR_SUBLINK fix.
Commits3f7323c et al missed the possibility that the Paramsthey are looking for could be buried under implicit coercions,as well as other stuff that processIndirection() could add tothe original targetlist entry. Copy the code in ruleutils.cthat deals with such cases. (I thought about refactoring sothat there's just one copy; but seeing that we only need thisin old back branches, it seems not worth the trouble.)Per off-list report from Andre Lin. As before, only v10-v13need the patch.Discussion:https://postgr.es/m/17596-c5357f61427a81dc@postgresql.org1 parentd7bc6ea commit18f5108
File tree
3 files changed
+54
-20
lines changed- src
- backend/optimizer/plan
- test/regress
- expected
- sql
3 files changed
+54
-20
lines changedLines changed: 37 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
914 | 914 |
| |
915 | 915 |
| |
916 | 916 |
| |
917 |
| - | |
| 917 | + | |
| 918 | + | |
| 919 | + | |
| 920 | + | |
918 | 921 |
| |
919 | 922 |
| |
920 | 923 |
| |
921 | 924 |
| |
922 | 925 |
| |
923 | 926 |
| |
| 927 | + | |
924 | 928 |
| |
925 | 929 |
| |
926 | 930 |
| |
927 | 931 |
| |
928 |
| - | |
| 932 | + | |
| 933 | + | |
| 934 | + | |
| 935 | + | |
| 936 | + | |
| 937 | + | |
| 938 | + | |
| 939 | + | |
| 940 | + | |
| 941 | + | |
| 942 | + | |
| 943 | + | |
| 944 | + | |
| 945 | + | |
| 946 | + | |
| 947 | + | |
| 948 | + | |
| 949 | + | |
| 950 | + | |
| 951 | + | |
| 952 | + | |
| 953 | + | |
| 954 | + | |
| 955 | + | |
| 956 | + | |
| 957 | + | |
| 958 | + | |
| 959 | + | |
| 960 | + | |
| 961 | + | |
| 962 | + | |
929 | 963 |
| |
930 |
| - | |
| 964 | + | |
931 | 965 |
| |
932 | 966 |
| |
933 | 967 |
| |
|
Lines changed: 12 additions & 12 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1717 | 1717 |
| |
1718 | 1718 |
| |
1719 | 1719 |
| |
1720 |
| - | |
| 1720 | + | |
1721 | 1721 |
| |
1722 | 1722 |
| |
1723 | 1723 |
| |
1724 | 1724 |
| |
1725 | 1725 |
| |
1726 | 1726 |
| |
1727 |
| - | |
1728 |
| - | |
| 1727 | + | |
| 1728 | + | |
1729 | 1729 |
| |
1730 |
| - | |
1731 |
| - | |
| 1730 | + | |
| 1731 | + | |
1732 | 1732 |
| |
1733 | 1733 |
| |
1734 | 1734 |
| |
1735 | 1735 |
| |
1736 |
| - | |
| 1736 | + | |
1737 | 1737 |
| |
1738 | 1738 |
| |
1739 | 1739 |
| |
1740 | 1740 |
| |
1741 |
| - | |
| 1741 | + | |
1742 | 1742 |
| |
1743 | 1743 |
| |
1744 |
| - | |
| 1744 | + | |
1745 | 1745 |
| |
1746 | 1746 |
| |
1747 | 1747 |
| |
1748 | 1748 |
| |
1749 | 1749 |
| |
1750 | 1750 |
| |
1751 |
| - | |
| 1751 | + | |
1752 | 1752 |
| |
1753 | 1753 |
| |
1754 |
| - | |
| 1754 | + | |
1755 | 1755 |
| |
1756 | 1756 |
| |
1757 | 1757 |
| |
1758 | 1758 |
| |
1759 | 1759 |
| |
1760 | 1760 |
| |
1761 |
| - | |
1762 |
| - | |
| 1761 | + | |
| 1762 | + | |
1763 | 1763 |
| |
1764 | 1764 |
| |
1765 | 1765 |
| |
|
Lines changed: 5 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
632 | 632 |
| |
633 | 633 |
| |
634 | 634 |
| |
635 |
| - | |
| 635 | + | |
636 | 636 |
| |
637 | 637 |
| |
638 | 638 |
| |
639 | 639 |
| |
640 | 640 |
| |
641 | 641 |
| |
642 | 642 |
| |
643 |
| - | |
644 |
| - | |
| 643 | + | |
| 644 | + | |
645 | 645 |
| |
646 |
| - | |
647 |
| - | |
| 646 | + | |
| 647 | + | |
648 | 648 |
| |
649 | 649 |
| |
650 | 650 |
| |
|
0 commit comments
Comments
(0)