Re: Logical replication timeout problem
Fabrice Chapuis <fabrice636861@gmail.com>
From: Fabrice Chapuis <fabrice636861@gmail.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-09-22T16:15:58Z
Lists: pgsql-hackers
If you would like I can test the patch you send to me. Regards Fabrice On Wed, Sep 22, 2021 at 11:02 AM Amit Kapila <amit.kapila16@gmail.com> wrote: > On Tue, Sep 21, 2021 at 9:12 PM Fabrice Chapuis <fabrice636861@gmail.com> > wrote: > > > > > IIUC, these are called after processing each WAL record so not > > sure how is it possible in your case that these are not reached? > > > > I don't know, as you say, to highlight the problem we would have to > debug the WalSndKeepaliveIfNecessary function > > > > > I was curious to know if the walsender has exited before walreceiver > > > > During the last tests we made we didn't observe any timeout of the wal > sender process. > > > > > Do you mean you are planning to change from 1 minute to 5 minutes? > > > > We set wal_sender_timeout/wal_receiver_timeout to 5' and launch new > test. The result is surprising and rather positive there is no timeout any > more in the log and the 20Gb of snap files are removed in less than 5 > minutes. > > How to explain that behaviour, why the snap files are consumed suddenly > so quickly. > > > > I think it is because we decide that the data in those snap files > doesn't need to be sent at xact end, so we remove them. > > > I choose the value arbitrarily for > wal_sender_timeout/wal_receiver_timeout parameters, are theses values > appropriate from your point of view? > > > > It is difficult to say what is the appropriate value for these > parameters unless in some way we debug WalSndKeepaliveIfNecessary() to > find why it didn't send keep alive when it is expected. Would you be > able to make code changes and test or if you want I can make changes > and send the patch if you can test it? If not, is it possible that in > some way you send a reproducible test? > > -- > With Regards, > Amit Kapila. >
Commits
-
Fix the logical replication timeout during large DDLs.
- 8c58624df462 16.0 landed
-
Fix the logical replication timeout during large transactions.
- f95d53eded55 15.0 landed
- d6da71fa8f28 14.4 landed
- 55558df23741 13.8 landed
- f832b5007c1c 12.12 landed
- 87c1dd246af8 11.17 landed
- a4015ec0375d 10.22 landed
-
Rethink the delay-checkpoint-end mechanism in the back-branches.
- 10520f434687 14.3 cited
-
Revert "Logical decoding of sequences"
- 2c7ea57e56ca 15.0 cited
-
Skip empty transactions for logical replication.
- d5a9d86d8ffc 15.0 cited
-
Allow specifying column lists for logical replication
- 923def9a533a 15.0 cited
-
Add decoding of sequences to built-in replication
- 75b1521dae1f 15.0 cited
-
Fix ABI break introduced by commit 4daa140a2f.
- 56e366f6757d 13.4 cited
-
Lag tracking for logical replication
- 024711bb5446 10.0 cited