Thanks. I have refactored the code for synchronize_one_slot() as there was some code-repetition. Please take it if you find it okay.
Ok for the goto retry.
Also I felt that when we create a slot through slot-synchronization, we should create it with allow_overwrite as false. And thus in the attached patch, I have changed that part as well.
The problem if allow_overwrite is set to false on the new standby it does not reflect the "allow_overwrite" parameter value set on the the primary instance. At the moment we can let the slot creation with allow_overwrite set to false.
It is a top up patch. Attached it as txt, please rename before applying atop your changes. Attached the steps for your reference.
On Tue, Sep 30, 2025 at 12:17 PM Fabrice Chapuis <fabrice636861@gmail.com> wrote: > > Hi, > > Here the generated v2 of the Patch. >
Thanks. I have refactored the code for synchronize_one_slot() as there was some code-repetition. Please take it if you find it okay.
Also I felt that when we create a slot through slot-synchronization, we should create it with allow_overwrite as false. And thus in the attached patch, I have changed that part as well.
It is a top up patch. Attached it as txt, please rename before applying atop your changes. Attached the steps for your reference.