Re: synchronized_standby_slots behavior inconsistent with quorum-based synchronous replication - Mailing list pgsql-hackers

Hi Japin,

> Thanks for updating the patch.  A minor nitpick:
>
> 1.
> +typedef enum
> +{
> +       SS_SLOT_NOT_FOUND,              /* slot does not exist */
> +       SS_SLOT_LOGICAL,                /* slot is logical, not physical */
> +       SS_SLOT_INVALIDATED,    /* slot has been invalidated */
> +       SS_SLOT_INACTIVE,               /* slot is inactive (standby not connected) */
> +       SS_SLOT_LAGGING                 /* slot exists and is active but has not caught up */
> +} SyncStandbySlotsState;
>
> IIRC, trailing commas are now used after the last enum.
>

Yes, I could see that in some of the lately added enums. Thanks for
raising this point, it has been addressed in the attached patch.

> 2.
> +       slot_states = (SyncStandbySlotsStateInfo *)
> +               palloc(sizeof(SyncStandbySlotsStateInfo) * synchronized_standby_slots_config->nslotnames);
>
> With palloc_array() now available, it would be preferable.
>

Makes sense. The attached patch addresses this too.

--
With Regards,
Ashutosh Sharma.