Re: Typo in tablesync comment - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Typo in tablesync comment
Date
Msg-id YBjgxXT987gyosEo@paquier.xyz
Whole thread Raw
In response to Typo in tablesync comment  (Peter Smith <smithpb2250@gmail.com>)
Responses Re: Typo in tablesync comment
Re: Typo in tablesync comment
List pgsql-hackers
On Tue, Feb 02, 2021 at 10:38:31AM +1100, Peter Smith wrote:
> PSA a trivial patch to correct what seems like a typo in the tablesync comment.

- *       subscribed tables and their state.  Some transient state during data
- *       synchronization is kept in shared memory.  The states SYNCWAIT and
+ *       subscribed tables and their state.  Some transient states during data
+ *       synchronization are kept in shared memory. The states SYNCWAIT and

This stuff refers to SUBREL_STATE_* in pg_subscription_rel.h, and FWIW
I find confusing the term "transient" in this context as a state may
last for a rather long time, depending on the time it takes to
synchronize the relation, no?  I am wondering if we could do better
here, say:
"The state tracking the progress of the relation synchronization is
additionally stored in shared memory, with SYNCWAIT and CATCHUP only
appearing in memory."
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Ajin Cherian
Date:
Subject: Re: Single transaction in the tablesync worker?
Next
From: Michael Paquier
Date:
Subject: Re: fix typo in reorderbuffer.c