On Wed, Mar 27, 2024 at 11:05 AM Bharath Rupireddy <bharath.rupireddyforpostg...@gmail.com> wrote: > > Fixed an issue in synchronize_slots where DatumGetLSN is being used in > place of DatumGetTimestampTz. Found this via CF bot member [1], not on > my dev system. > > Please find the attached v6 patch.
Thanks for the patch. Few trivial things: ---------- 1) system-views.sgml: a) "Note that the slots" --> "Note that the slots on the standbys," --it is good to mention "standbys" as synced could be true on primary as well (promoted standby) b) If you plan to add more info which Bertrand suggested, then it will be better to make a <note> section instead of using "Note" 2) commit msg: "The impact of this on a promoted standby inactive_since is always NULL for all synced slots even after server restart. " Sentence looks broken. --------- Apart from the above trivial things, v26-001 looks good to me. thanks Shveta
