pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
Subject: [PATCH 2/2] Fix typo
Date: Mon, 20 May 2024 18:01:45 +0200

Confusion induced by change of meaning of "omit/include" flag for
pending-detach partitions during development.

Noted by Tender Wang.
---
 src/backend/executor/execPartition.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


inline patch 1: src/backend/executor/execPartition.c (+1/-1 lines) download inline-1.patchdiff --git a/src/backend/executor/execPartition.c b/src/backend/executor/execPartition.c
index 637e5bac5d..1184ca4096 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);
-- 
2.39.2



--2d3ceazxqldvimr4--





Message-ID: <no-message-id-7773@localhost>
Permalink:  ../no-message-id-7773@localhost/
Also on:    postgresql.org/message-id/no-message-id-7773@localhost

 ·  · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: alvherre@alvh.no-ip.org
  Subject: Re: [PATCH 2/2] Fix typo
  In-Reply-To: <no-message-id-7773@localhost>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox