On Tue, Sep 12, 2023 at 01:22:50PM +0000, Zhijie Hou (Fujitsu) wrote: > +/* > + * binary_upgrade_sub_replication_origin_advance > + * > + * Update the remote_lsn for the subscriber's replication origin. > + */ > +Datum > +binary_upgrade_sub_replication_origin_advance(PG_FUNCTION_ARGS) > +{ > > Is there any usage apart from pg_upgrade for this function, if not, I think > we'd better move this function to pg_upgrade_support.c. If yes, I think maybe > better to rename it to a general one.
I was equally surprised by the choice of the patch regarding the location of these functions, so I agree with your point that these functions should be in pg_upgrade_support.c. All the sub-routines these two functions rely on are defined in some headers already, so there seem to be nothing new required for pg_upgrade_support.c. -- Michael
signature.asc
Description: PGP signature