0002-Minor-edits-20240322.patch

text/x-patch

Filename: 0002-Minor-edits-20240322.patch
Type: text/x-patch
Part: 0
Message: Re: Memory consumed by child SpecialJoinInfo in partitionwise join planning

Patch

Format: format-patch
Series: patch 0002
Subject: Minor edits
File+
src/backend/optimizer/path/joinrels.c 5 5
From 555cb9825eaa784d23c80c28d2cee27570cad781 Mon Sep 17 00:00:00 2001
From: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
Date: Fri, 22 Mar 2024 15:39:08 +0530
Subject: [PATCH 2/2] Minor edits

Ashutosh Bapat
---
 src/backend/optimizer/path/joinrels.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/backend/optimizer/path/joinrels.c b/src/backend/optimizer/path/joinrels.c
index bef5b5f213..5d055be855 100644
--- a/src/backend/optimizer/path/joinrels.c
+++ b/src/backend/optimizer/path/joinrels.c
@@ -659,10 +659,10 @@ join_is_legal(PlannerInfo *root, RelOptInfo *rel1, RelOptInfo *rel2,
  *    Populate the given SpecialJoinInfo for a plain inner join between rel1
  *    and rel2
  *
- * Inner joins do not normally have an associated SpecialJoinInfo node.  But
- * some functions involved in join planning require one to be passed,
- * containing at least the information of which relations are being joined,
- * so we initialize that information here.
+ * Normally, an inner join does not have a SpecialJoinInfo node associated with
+ * it. But some functions involved in join planning require one containing at
+ * least the information of which relations are being joined.  So we initialize
+ * that information here.
  */
 void
 init_dummy_sjinfo(SpecialJoinInfo *sjinfo, Relids left_relids,
@@ -1686,7 +1686,7 @@ try_partitionwise_join(PlannerInfo *root, RelOptInfo *rel1, RelOptInfo *rel2,
  * are the relids of left and right side of the join respectively.
  *
  * If translations are added to or removed from this function, consider
- * updating free_child_join_sjinfo() appropriately.
+ * updating free_child_join_sjinfo() accordingly.
  */
 static SpecialJoinInfo *
 build_child_join_sjinfo(PlannerInfo *root, SpecialJoinInfo *parent_sjinfo,
-- 
2.25.1