postgres.git / summary / log / commit / refs

commit    a2dff271ebe2a0547d46e90dcb02c088cf2f031c
Author:   Alvaro Herrera <alvherre@alvh.no-ip.org>
Date:     Thu Jun 27 17:51:47 2024 +0000
First released in: 17.0, 14.13, 15.8, 16.4

    Fix thinkos in comments
    
    The first one was noticed by Tender Wang and introduced with
    8aba9322511f; the other one was newly introduced with dbca3469ebf8.


 src/backend/executor/execPartition.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/backend/executor/execPartition.c b/src/backend/executor/execPartition.c
index c37a19a91f4..76518862291 100644
--- a/src/backend/executor/execPartition.c
+++ b/src/backend/executor/execPartition.c
@@ -1871,7 +1871,7 @@ CreatePartitionPruneState(PlanState *planstate, PartitionPruneInfo *pruneinfo)
 	int			i;
 	ExprContext *econtext = planstate->ps_ExprContext;
 
-	/* For data reading, executor always omits detached partitions */
+	/* For data reading, executor always includes detached partitions */
 	if (estate->es_partition_directory == NULL)
 		estate->es_partition_directory =
 			CreatePartitionDirectory(estate->es_query_cxt, false);
@@ -1954,8 +1954,8 @@ CreatePartitionPruneState(PlanState *planstate, PartitionPruneInfo *pruneinfo)
 			 * this by creating new subplan_map and subpart_map arrays that
 			 * corresponds to the ones in the PruneInfo where the new
 			 * partition descriptor's OIDs match.  Any that don't match can be
-			 * set to -1, as if they were pruned.  Both arrays must be in
-			 * numerical OID order.
+			 * set to -1, as if they were pruned.  By construction, both
+			 * arrays are in partition bounds order.
 			 */
 			pprune->nparts = partdesc->nparts;
 			pprune->subplan_map = palloc(sizeof(int) * partdesc->nparts);


[parent: 3e53492aa708]