Dear Bertrand,

> s/to avoid the seeing a xl_running_xacts/to avoid seeing a xl_running_xacts/?

Fixed.

> 
> === 2 (Nit)
> 
> /* For testing slot invalidation due to the conflict */
> 
> Not sure "due to the conflict" is needed.
>

OK, removed.

> ==== About PG17-v2-0001
> 
> === 3
> 
> The commit message still mentions injection point.

Oh, removed.

> === 4
> 
> -# Note that pg_current_snapshot() is used to get the horizon.  It does
> -# not generate a Transaction/COMMIT WAL record, decreasing the risk of
> -# seeing a xl_running_xacts that would advance an active replication slot's
> -# catalog_xmin.  Advancing the active replication slot's catalog_xmin
> -# would break some tests that expect the active slot to conflict with
> -# the catalog xmin horizon.
> 
> I'd be tempted to not remove this comment but reword it a bit instead. 
> Something
> like?
> 
> # Note that pg_current_snapshot() is used to get the horizon.  It does
> # not generate a Transaction/COMMIT WAL record, decreasing the risk of
> # seeing a xl_running_xacts that would advance an active replication slot's
> # catalog_xmin.  Advancing the active replication slot's catalog_xmin
> # would break some tests that expect the active slot to conflict with
> # the catalog xmin horizon. We ensure that active replication slots are not
> # created for tests that might produce this race condition though.

Added.

> === 6 (Nit)
> 
> In drop_logical_slots(), s/needs_active_slot/drop_active_slot/?

Fixed.

> === 7 (Nit)
> 
> In check_slots_conflict_reason(), s/needs_active_slot/checks_active_slot/?

Fixed.

> ==== About PG16-v2-0001
> 
> Same as for PG17-v2-0001.

I followed all needed changes.

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Attachment: v3-0001-Stabilize-035_standby_logical_decoding.pl-by-usin.patch
Description: v3-0001-Stabilize-035_standby_logical_decoding.pl-by-usin.patch

Attachment: PG16-v3-0001-Stabilize-035_standby_logical_decoding.pl-by-usin.patch
Description: PG16-v3-0001-Stabilize-035_standby_logical_decoding.pl-by-usin.patch

Attachment: PG17-v3-0001-Stabilize-035_standby_logical_decoding.pl-by-usin.patch
Description: PG17-v3-0001-Stabilize-035_standby_logical_decoding.pl-by-usin.patch

Reply via email to