On Wed, Sep 21, 2022 at 8:22 PM Peter Smith <smithpb2...@gmail.com> wrote: > ... > > > On Wed, Sep 21, 2022 at 3:23 PM Amit Kapila <amit.kapil...@gmail.com> > > > wrote: > > > ... > > > > > > > Can't we use the existing function ReplicationOriginNameForTablesync() > > > > by passing relid as InvalidOid for this purpose? We need a check > > > > inside to decide which name to construct, otherwise, it should be > > > > fine. If we agree with this, then we can change the name of the > > > > function to something like ReplicationOriginNameForLogicalRep or > > > > ReplicationOriginNameForLogicalRepWorkers. > > > > ... > > OK, I can unify the 2 functions as you suggested. I will post another > patch in a few days. >
PSA patch v3 to combine the different replication origin name formatting in a single function ReplicationOriginNameForLogicalRep as suggested. ------ Kind Regards, Peter Smith. Fujitsu Australia
v3-0001-Add-common-function-ReplicationOriginNameForLogic.patch
Description: Binary data
