Re: walsender.c fileheader comment

2024-07-19 Thread Tomas Vondra
On 7/19/24 07:02, Peter Smith wrote: > ... >> >> To conclude, I think this probably makes the comments more confusing. If >> we want to make it clearer, I'd probably start by clarifying what the >> "stopping" state means. Also, it's a bit surprising we may not actually >> go through the "stopping"

Re: walsender.c fileheader comment

2024-07-19 Thread Peter Smith
Hi, Thankyou for taking the time to look at this and reply. > > I did look at this, and while the explanation in the current comment may > seem a bit confusing, I'm not sure the suggested changes improve the > situation very much. > > This suggests the two comments somehow disagree, but it does no

Re: walsender.c fileheader comment

2024-07-16 Thread Tomas Vondra
On 6/11/24 04:35, Peter Smith wrote: > Hi, > > I was reading the walsender.c fileheader comment while studying > another thread. I think if there is logical replication in progress > then the PROCSIG_WALSND_INIT_STOPPING handler will *always* switch to > a &qu

walsender.c fileheader comment

2024-06-10 Thread Peter Smith
Hi, I was reading the walsender.c fileheader comment while studying another thread. I think if there is logical replication in progress then the PROCSIG_WALSND_INIT_STOPPING handler will *always* switch to a "stopping" state: e.g., /* * Handle PROCSIG_WALSND_INIT_STOPPING signal