diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-06-02 11:52:35 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-06-02 11:52:35 -0400 |
commit | 79c50ca57828e9f8375766b36cce1e2960eebf87 (patch) | |
tree | 42d14747fcec6ce20aafb6bdd15a07f388fba92c /src | |
parent | 9e3b3ff2664dd0b349d2a6d6f047128cb3489cf2 (diff) | |
download | postgresql-79c50ca57828e9f8375766b36cce1e2960eebf87.tar.gz postgresql-79c50ca57828e9f8375766b36cce1e2960eebf87.zip |
Update plannodes.h's comments about PlanRowMark.
The reference here to different physical column numbers in inherited
UPDATE/DELETE plans is obsolete as of 86dc90056; remove it. Also
rework the text about inheritance cases to make it clearer.
Diffstat (limited to 'src')
-rw-r--r-- | src/include/nodes/plannodes.h | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/include/nodes/plannodes.h b/src/include/nodes/plannodes.h index 57f320a1759..aaa3b65d049 100644 --- a/src/include/nodes/plannodes.h +++ b/src/include/nodes/plannodes.h @@ -1086,9 +1086,9 @@ typedef enum RowMarkType * When the planner discovers that a relation is the root of an inheritance * tree, it sets isParent true, and adds an additional PlanRowMark to the * list for each child relation (including the target rel itself in its role - * as a child). isParent is also set to true for the partitioned child - * relations, which are not scanned just like the root parent. The child - * entries have rti == child rel's RT index and prti == parent's RT index, + * as a child, if it is not a partitioned table). Any non-leaf partitioned + * child relations will also have entries with isParent = true. The child + * entries have rti == child rel's RT index and prti == top parent's RT index, * and can therefore be recognized as children by the fact that prti != rti. * The parent's allMarkTypes field gets the OR of (1<<markType) across all * its children (this definition allows children to use different markTypes). @@ -1109,8 +1109,7 @@ typedef enum RowMarkType * means we needn't renumber rowmarkIds when flattening subqueries, which * would require finding and renaming the resjunk columns as well.) * Note this means that all tables in an inheritance hierarchy share the - * same resjunk column names. However, in an inherited UPDATE/DELETE the - * columns could have different physical column numbers in each subplan. + * same resjunk column names. */ typedef struct PlanRowMark { |