diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-02-07 11:56:43 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-02-07 11:56:43 -0500 |
| commit | 2cbbffff05b853733ae19ca7047cfeb9a346a497 (patch) | |
| tree | 45cab0c8d474b288b4d6c6d91cb100f52752726d /src/backend | |
| parent | aa6954104644334c53838f181053b9f7aa13f58c (diff) | |
| download | postgresql-2cbbffff05b853733ae19ca7047cfeb9a346a497.tar.gz postgresql-2cbbffff05b853733ae19ca7047cfeb9a346a497.zip | |
Remove leftover code in deconstruct_distribute_oj_quals().
The initial "put back OJ relids" adjustment of ojscope was
incorrect and unnecessary; it seems to be a leftover from
when I (tgl) was trying to get this function to work at all.
Richard Guo
Discussion: https://postgr.es/m/CAMbWs4-L2C47ZGZPabBAi5oDZsKmsbvhYcGCy5o=gCjsaG_ZQA@mail.gmail.com
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/optimizer/plan/initsplan.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/backend/optimizer/plan/initsplan.c b/src/backend/optimizer/plan/initsplan.c index 2d9aa85e591..904f710d593 100644 --- a/src/backend/optimizer/plan/initsplan.c +++ b/src/backend/optimizer/plan/initsplan.c @@ -1903,13 +1903,6 @@ deconstruct_distribute_oj_quals(PlannerInfo *root, int save_last_rinfo_serial; ListCell *lc; - /* - * Put any OJ relids that were removed from min_righthand back into - * ojscope, else distribute_qual_to_rels will complain. - */ - ojscope = bms_join(ojscope, bms_intersect(sjinfo->commute_below, - sjinfo->syn_righthand)); - /* Identify the outer joins this one commutes with */ joins_above = sjinfo->commute_above_r; joins_below = bms_intersect(sjinfo->commute_below, |