fix a comment
Hi, Hackers:
In function ExecGetTriggerResultRel, we can see comments:
/* First, search through the query result relations */ ...
/*
* Third, search through the result relations that were created during
* tuple routing, if any.
*/
But the 'Second' was deleted since commit 1375422c78.
Update the 'Third' to 'Second', please see the attachment.
Thoughts?
Best wishes
Yukun Wang
Attachments:
fix-a-typo.diffapplication/octet-stream; name=fix-a-typo.diffDownload
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index 8d0f3de76e..bb75d907a3 100644
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -1295,7 +1295,7 @@ ExecGetTriggerResultRel(EState *estate, Oid relid)
}
/*
- * Third, search through the result relations that were created during
+ * Second, search through the result relations that were created during
* tuple routing, if any.
*/
foreach(l, estate->es_tuple_routing_result_relations)
On Fri, Apr 23, 2021 at 12:12 PM wangyukun@fujitsu.com
<wangyukun@fujitsu.com> wrote:
Hi, Hackers:
In function ExecGetTriggerResultRel, we can see comments:
/* First, search through the query result relations */ ...
/*
* Third, search through the result relations that were created during
* tuple routing, if any.
*/But the 'Second' was deleted since commit 1375422c78.
Update the 'Third' to 'Second', please see the attachment.
Thoughts?
Well yes, looks good to me.
How about simply removing these numbering?
Regards,
Amul
Hi, Amul
Thank you for reviewing.
How about simply removing these numbering?
Agree. Please see the v2 patch which delete the number in comment.
Best wishes
Yukun Wang
-----Original Message-----
From: Amul Sul <sulamul@gmail.com>
Sent: Friday, April 23, 2021 3:51 PM
To: Wang, Yukun/王 俞坤 <wangyukun@fujitsu.com>
Cc: pgsql-hackers@postgresql.org
Subject: Re: fix a comment
On Fri, Apr 23, 2021 at 12:12 PM wangyukun@fujitsu.com <wangyukun@fujitsu.com> wrote:
Hi, Hackers:
In function ExecGetTriggerResultRel, we can see comments:
/* First, search through the query result relations */ ...
/*
* Third, search through the result relations that were created
during
* tuple routing, if any.
*/But the 'Second' was deleted since commit 1375422c78.
Update the 'Third' to 'Second', please see the attachment.
Thoughts?
Well yes, looks good to me.
How about simply removing these numbering?
Regards,
Amul
Attachments:
fix-a-comment.diffapplication/octet-stream; name=fix-a-comment.diffDownload
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index 8d0f3de76e..bbab991e0b 100644
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -1286,7 +1286,7 @@ ExecGetTriggerResultRel(EState *estate, Oid relid)
Relation rel;
MemoryContext oldcontext;
- /* First, search through the query result relations */
+ /* Search through the query result relations */
foreach(l, estate->es_opened_result_relations)
{
rInfo = lfirst(l);
@@ -1295,7 +1295,7 @@ ExecGetTriggerResultRel(EState *estate, Oid relid)
}
/*
- * Third, search through the result relations that were created during
+ * Search through the result relations that were created during
* tuple routing, if any.
*/
foreach(l, estate->es_tuple_routing_result_relations)
On Fri, Apr 23, 2021 at 07:03:40AM +0000, wangyukun@fujitsu.com wrote:
Agree. Please see the v2 patch which delete the number in comment.
Indeed, this set of comments became a bit obsolete after 1375422, as
you saied upthread. This simplification looks fine to me, so
applied. I am in a mood for such patches since yesterday..
--
Michael
On Sat, Apr 24, 2021 at 11:43 AM Michael Paquier <michael@paquier.xyz> wrote:
On Fri, Apr 23, 2021 at 07:03:40AM +0000, wangyukun@fujitsu.com wrote:
Agree. Please see the v2 patch which delete the number in comment.
Indeed, this set of comments became a bit obsolete after 1375422, as
you saied upthread. This simplification looks fine to me, so
applied. I am in a mood for such patches since yesterday..
:)
Thank you !
Regards,
Amul