| From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
|---|---|
| To: | Shubham Khanna <khannashubham1197(at)gmail(dot)com> |
| Cc: | vignesh C <vignesh21(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Log a warning in pg_createsubscriber for max_slot_wal_keep_size |
| Date: | 2025-01-05 21:52:07 |
| Message-ID: | CAHut+PuAB6Op8kW4vbfOGbJp-AdZpBc2LRtKEnBNArky-JPx2w@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Shubham.
The patch v6-0001 LGTM.
OTOH, if you want to be picky, the docs wording could be slightly
modified to be more consistent with the coded warning message.
CURRENT
Replication failures can occur if required WAL files are prematurely
deleted. To prevent this, the source server must set <xref
linkend="guc-max-slot-wal-keep-size"/> to <literal>-1</literal>,
ensuring WAL files are not automatically removed.
SUGGESTION
Replication failures can occur if required WAL files are missing. To
prevent this, the source server must set <xref
linkend="guc-max-slot-wal-keep-size"/> to <literal>-1</literal> to
ensure that required WAL files are not prematurely removed.
~~~
See the attached NITPICKS diff if you want to make this change.
======
Kind Regards,
Peter Smith.
Fujitsu Australia
| Attachment | Content-Type | Size |
|---|---|---|
| PS_NITPICKS_v60001.txt | text/plain | 845 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Paul Jungwirth | 2025-01-05 23:20:07 | Re: Index AM API cleanup |
| Previous Message | jian he | 2025-01-05 16:10:37 | Re: Re: proposal: schema variables |