Comment typo in CheckCmdReplicaIdentity

Started by Peter Smithalmost 4 years ago5 messages
#1Peter Smith
smithpb2250@gmail.com
1 attachment(s)

PSA patch to fix a comment typo.

(The 'OR' should not be uppercase - that keyword is irrelevant here).

------
Kind Regards,
Peter Smith.
Fujitsu Australia

Attachments:

v1-0001-Fix-comment-typo-CheckCmdReplicaIdentity.patchapplication/octet-stream; name=v1-0001-Fix-comment-typo-CheckCmdReplicaIdentity.patchDownload
From b646f4500eb31743600df6f69181211584f5f16d Mon Sep 17 00:00:00 2001
From: Peter Smith <peter.b.smith@fujitsu.com>
Date: Mon, 7 Mar 2022 09:15:14 +1100
Subject: [PATCH v1] Fix comment typo CheckCmdReplicaIdentity

---
 src/backend/executor/execReplication.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/executor/execReplication.c b/src/backend/executor/execReplication.c
index de106d7..09f78f2 100644
--- a/src/backend/executor/execReplication.c
+++ b/src/backend/executor/execReplication.c
@@ -607,7 +607,7 @@ CheckCmdReplicaIdentity(Relation rel, CmdType cmd)
 		return;
 
 	/*
-	 * This is either UPDATE OR DELETE and there is no replica identity.
+	 * This is UPDATE/DELETE and there is no replica identity.
 	 *
 	 * Check if the table publishes UPDATES or DELETES.
 	 */
-- 
1.8.3.1

#2Michael Paquier
michael@paquier.xyz
In reply to: Peter Smith (#1)
Re: Comment typo in CheckCmdReplicaIdentity

On Mon, Mar 07, 2022 at 09:31:33AM +1100, Peter Smith wrote:

PSA patch to fix a comment typo.

(The 'OR' should not be uppercase - that keyword is irrelevant here).

I was looking at the whole routine, and your suggestion looks like an
improvement to me. Will apply if there are no objections.
--
Michael

#3Julien Rouhaud
rjuju123@gmail.com
In reply to: Michael Paquier (#2)
Re: Comment typo in CheckCmdReplicaIdentity

On Mon, Mar 07, 2022 at 10:36:24AM +0900, Michael Paquier wrote:

On Mon, Mar 07, 2022 at 09:31:33AM +1100, Peter Smith wrote:

PSA patch to fix a comment typo.

(The 'OR' should not be uppercase - that keyword is irrelevant here).

I was looking at the whole routine, and your suggestion looks like an
improvement to me. Will apply if there are no objections.

+1

#4Michael Paquier
michael@paquier.xyz
In reply to: Julien Rouhaud (#3)
Re: Comment typo in CheckCmdReplicaIdentity

On Mon, Mar 07, 2022 at 10:28:08AM +0800, Julien Rouhaud wrote:

+1

And done.
--
Michael

#5Peter Smith
smithpb2250@gmail.com
In reply to: Michael Paquier (#4)
Re: Comment typo in CheckCmdReplicaIdentity

On Tue, Mar 8, 2022 at 4:31 PM Michael Paquier <michael@paquier.xyz> wrote:

On Mon, Mar 07, 2022 at 10:28:08AM +0800, Julien Rouhaud wrote:

+1

And done.
--
Michael

Thanks!

------
Kind Regards,
Peter Smith.
Fujitsu Australia