Fix obsolete comments referencing JoinPathExtraData.extra_lateral_rels.
authorTom Lane
Wed, 14 Apr 2021 18:28:17 +0000 (14:28 -0400)
committerTom Lane
Wed, 14 Apr 2021 18:28:24 +0000 (14:28 -0400)
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

src/backend/optimizer/path/joinpath.c

index e9b6968b1d7cdc163ef101722b36869ca40ad823..4c30c6556409d602bf92a99024febdd5bb060a05 100644 (file)
@@ -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.
     */