Re: Exit walsender before confirming remote flush in logical replication

Hi Fujii-san,

Thank you for the testing.

On 3/25/26 15:39, Fujii Masao wrote:
 > I tested wal_sender_shutdown_timeout under several configurations and
 > encountered a case where the primary shutdown got stuck, even with the patch
 > and wal_sender_shutdown_timeout = 1. I'm not sure yet whether this is a bug in
 > the patch or an issue with my test setup, but anyway I'd like to share
 > the reproduction steps for reference.

It seems that the problem lies in the logic of calculating sleep time in
WalSndComputeSleeptime function. If the parameter wal_sender_timeout is set to
one hour and the function WalSndWait executes with an argument sleeptime = 1h,
then the variable shutdown_request_timestamp will only be updated after one
hour at next call of WalSndCheckShutdownTimeout immediately following the
waiting period completion.

May be to use the minimal timeout in WalSndComputeSleeptimes or to use
the timeouts mechanism (timeout.c), but WalSndWait should wake up on latch then.

With best regards,
Vitaly