On Fri, Nov 21, 2025 at 6:21 PM Shlok Kyal <[email protected]> wrote: > > The Cbot complained that it was not able to build the docs. I have > fixed it and attached the latest patch. >
Few comments on 001: 1) In pgstat_report_replslotsync_skip(), shall we have a sanity check to ensure that slot is logical and function is called on a standby(RecoveryInProgress)? 2) In update_and_persist_local_synced_slot(), we have comments at 3 places to indicate that stats are updated in some other function.Instead, shall we have a generic comment in the header of this function? 3) Shall we have the test moved to the existing file 040_standby_failover_slots_sync? 4) We should be able to make test work without injection point, please try for that. Also it should be enough to test stats for one flow instead of multiple flows. thanks Shveta
