diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-04-14 14:28:17 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-04-14 14:28:24 -0400 |
| commit | e1623b7d86812ee6ab98a42f38b43a192c149988 (patch) | |
| tree | 7e04e31e79215b2f417437846d5bf82d2622d8fb /src | |
| parent | 9acaf1a62197205b06a85afbfcaa7ffaac939ef3 (diff) | |
| download | postgresql-e1623b7d86812ee6ab98a42f38b43a192c149988.tar.gz postgresql-e1623b7d86812ee6ab98a42f38b43a192c149988.zip | |
Fix obsolete comments referencing JoinPathExtraData.extra_lateral_rels.
That field went away in commit edca44b15, but it seems that
commit 45be99f8c re-introduced some comments mentioning it.
Noted by James Coleman, though this isn't exactly his
proposed new wording. Also thanks to Justin Pryzby for
software archaeology.
Discussion: https://postgr.es/m/CAAaqYe8fxZjq3na+XkNx4C78gDqykH-7dbnzygm9Qa9nuDTePg@mail.gmail.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/optimizer/path/joinpath.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c index e9b6968b1d7..4c30c655640 100644 --- a/src/backend/optimizer/path/joinpath.c +++ b/src/backend/optimizer/path/joinpath.c @@ -670,8 +670,8 @@ try_partial_nestloop_path(PlannerInfo *root, /* * If the inner path is parameterized, the parameterization must be fully * satisfied by the proposed outer path. Parameterized partial paths are - * not supported. The caller should already have verified that no - * extra_lateral_rels are required here. + * not supported. The caller should already have verified that no lateral + * rels are required here. */ Assert(bms_is_empty(joinrel->lateral_relids)); if (inner_path->param_info != NULL) @@ -984,8 +984,8 @@ try_partial_hashjoin_path(PlannerInfo *root, /* * If the inner path is parameterized, the parameterization must be fully * satisfied by the proposed outer path. Parameterized partial paths are - * not supported. The caller should already have verified that no - * extra_lateral_rels are required here. + * not supported. The caller should already have verified that no lateral + * rels are required here. */ Assert(bms_is_empty(joinrel->lateral_relids)); if (inner_path->param_info != NULL) @@ -1714,7 +1714,7 @@ match_unsorted_outer(PlannerInfo *root, * partial path and the joinrel is parallel-safe. However, we can't * handle JOIN_UNIQUE_OUTER, because the outer path will be partial, and * therefore we won't be able to properly guarantee uniqueness. Nor can - * we handle extra_lateral_rels, since partial paths must not be + * we handle joins needing lateral rels, since partial paths must not be * parameterized. Similarly, we can't handle JOIN_FULL and JOIN_RIGHT, * because they can produce false null extended rows. */ |