On Mon, Feb 10, 2025 at 6:12 PM vignesh C <vignes...@gmail.com> wrote:
>
> On Mon, 10 Feb 2025 at 17:33, Nisha Moond <nisha.moond...@gmail.com> wrote:
> >
> > Here are the v73 patches incorporating the comments above and the
> > subsequent comments from [1].
> > - patch 002 is rebased on 001 with no new changes.
>
> Few comments:
> 1) For some reason SlotInvalidationCauses was with PGDLLIMPORT, this
> is removed now. This is required if it needs to be accessible by
> loaded modules. Is there any impact or is it ok?
> -extern PGDLLIMPORT const char *const SlotInvalidationCauses[];
> +typedef struct SlotInvalidationCauseMap
> +{
> + int cause;
> + const char *cause_name;
> +} SlotInvalidationCauseMap;
>
> 2) The new structure should be added to typedefs.list:
> +typedef struct SlotInvalidationCauseMap
> +{
> + int cause;
> + const char *cause_name;
> +} SlotInvalidationCauseMap;
>
> 3) After adding you can run pgindent on slot.h to indent the following code:
> +typedef struct SlotInvalidationCauseMap
> +{
> + int cause;
> + const char *cause_name;
> +} SlotInvalidationCauseMap;
>Addressed above comments, please find the attached v74 patches. Also, corrected the type of parameter "possible_causes" to 'uint32' in InvalidateObsoleteReplicationSlots() and InvalidatePossiblyObsoleteSlot(). -- Thanks, Nisha
v74-0001-Introduce-inactive_timeout-based-replication-slo.patch
Description: Binary data
v74-0002-Add-TAP-test-for-slot-invalidation-based-on-inac.patch
Description: Binary data
