Consistently use the term 'partitioned rel' in partprune comments

We were using 'partition rel' in a few places, which is quite confusing.

Author: Amit Langote
Reviewed-by: David Rowley
Reviewed-by: Michaël Paquier
Discussion: https://postgr.es/m/fd256561-31a2-4b7e-cd84-d8241e7ebc3f@lab.ntt.co.jp

diff --git a/src/backend/partitioning/partprune.c b/src/backend/partitioning/partprune.c
index 0a76b77d3efb6b313d3452d2cc5d2b4b44fe8ecb..cdc61a89974b0acf7620d576065ba6b2c80d8886 100644 (file)
--- a/src/backend/partitioning/partprune.c
+++ b/src/backend/partitioning/partprune.c
@@ -176,7 +176,7 @@ static bool partkey_datum_from_expr(PartitionPruneContext *context,
 
 /*
  * make_partition_pruneinfo
- *     Build List of PartitionPruneInfos, one for each 'partitioned_rels'.
+ *     Build List of PartitionPruneInfos, one for each partitioned rel.
  *     These can be used in the executor to allow additional partition
  *     pruning to take place.
  *
@@ -190,7 +190,7 @@ static bool partkey_datum_from_expr(PartitionPruneContext *context,
  * pruning done during planning will have pruned everything that can be.
  */
 List *
-make_partition_pruneinfo(PlannerInfo *root, List *partition_rels,
+make_partition_pruneinfo(PlannerInfo *root, List *partitioned_rels,
                         List *subpaths, List *prunequal)
 {
    RelOptInfo *targetpart = NULL;
@@ -229,11 +229,11 @@ make_partition_pruneinfo(PlannerInfo *root, List *partition_rels,
 
    /*
     * relid_subpart_map maps relid of a non-leaf partition to the index in
-    * 'partition_rels' of that rel (which will also be the index in the
+    * 'partitioned_rels' of that rel (which will also be the index in the
     * returned PartitionPruneInfo list of the info for that partition).
     */
    i = 1;
-   foreach(lc, partition_rels)
+   foreach(lc, partitioned_rels)
    {
        Index       rti = lfirst_int(lc);
 
@@ -246,8 +246,8 @@ make_partition_pruneinfo(PlannerInfo *root, List *partition_rels,
        relid_subpart_map[rti] = i++;
    }
 
-   /* We now build a PartitionPruneInfo for each rel in partition_rels */
-   foreach(lc, partition_rels)
+   /* We now build a PartitionPruneInfo for each partitioned rel */
+   foreach(lc, partitioned_rels)
    {
        Index       rti = lfirst_int(lc);
        RelOptInfo *subpart = find_base_rel(root, rti);

diff --git a/src/include/partitioning/partprune.h b/src/include/partitioning/partprune.h
index 09147b532c816dff80976f8844d67134d78afc7a..9944d2832f457dce404bc8de25fba31b200ae23c 100644 (file)
--- a/src/include/partitioning/partprune.h
+++ b/src/include/partitioning/partprune.h
@@ -74,7 +74,8 @@ typedef struct PartitionPruneContext
 #define PruneCxtStateIdx(partnatts, step_id, keyno) \
    ((partnatts) * (step_id) + (keyno))
 
-extern List *make_partition_pruneinfo(PlannerInfo *root, List *partition_rels,
+extern List *make_partition_pruneinfo(PlannerInfo *root,
+                        List *partitioned_rels,
                         List *subpaths, List *prunequal);
 extern Relids prune_append_rel_partitions(RelOptInfo *rel);
 extern Bitmapset *get_matching_partitions(PartitionPruneContext *context,