trivial comment fix

Started by Euler Taveiraover 3 years ago2 messages

euler@eulerto.com

over 3 years ago

1 attachment(s)

Hi,

While reading worker.c, I noticed that the referred SQL command was wrong.
ALTER SUBSCRIPTION ... REFRESH PUBLICATION instead of ALTER TABLE ... REFRESH
PUBLICATION. Trivial fix attached.

--
Euler Taveira
EDB https://www.enterprisedb.com/

Attachments:

fix.patchtext/x-patch; name=fix.patchDownload

diff --git a/src/backend/replication/logical/worker.c b/src/backend/replication/logical/worker.c
index 4171371296..7da7823c35 100644
--- a/src/backend/replication/logical/worker.c
+++ b/src/backend/replication/logical/worker.c
@@ -93,7 +93,7 @@
  * ReorderBufferFinishPrepared.
  *
  * If the subscription has no tables then a two_phase tri-state PENDING is
- * left unchanged. This lets the user still do an ALTER TABLE REFRESH
+ * left unchanged. This lets the user still do an ALTER SUBSCRIPTION REFRESH
  * PUBLICATION which might otherwise be disallowed (see below).
  *
  * If ever a user needs to be aware of the tri-state value, they can fetch it

John Naylor

john.naylor@enterprisedb.com

over 3 years ago

In reply to: Euler Taveira (#1)

Re: trivial comment fix

On Thu, Apr 28, 2022 at 7:27 AM Euler Taveira <euler@eulerto.com> wrote:

Hi,

While reading worker.c, I noticed that the referred SQL command was wrong.
ALTER SUBSCRIPTION ... REFRESH PUBLICATION instead of ALTER TABLE ... REFRESH
PUBLICATION. Trivial fix attached.

Pushed, thanks!

--
John Naylor
EDB: http://www.enterprisedb.com