On Thu, 20 Feb 2025 at 09:39, Nisha Moond <nisha.moond...@gmail.com> wrote: > > On Wed, Feb 19, 2025 at 12:40 PM Shlok Kyal <shlok.kyal....@gmail.com> wrote: > > Hi, > > > > The failover option is set to false by default while copying of the > > slots as it may cause some issue during slot synchronization as per > > discussion [1]. > > > > I have created a patch to update the documentation for the same. > > > > Thanks for the patch. > A couple of comments regarding the sentence: > > + not copied and set as <literal>false</literal> by default due to > + potential issues with the slot synchronization. > > 1) / and set as/ and is set to > "set to" is more natural and clearer when assigning a specific value > compared to "set as." > > 2) "due to potential issues..." wording implies that it is addressing > existing issues. I feel it would be better to say: > "... <literal>false</literal> by default to avoid potential issues > with the slot synchronization." >
Hi Nisha, I have addressed the comments and attached the updated v2 patch. Thanks and Regards, Shlok Kyal
v2-0001-Improve-documentation-for-pg_copy_logical_replica.patch
Description: Binary data
