Content-Type: text/plain;
	charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3826.700.81\))
Subject: Re: Allow GUC settings in CREATE SUBSCRIPTION CONNECTION to take
 effect
From: Chao Li <li.evan.chao@gmail.com>
In-Reply-To: 
 <CAHGQGwGnkM8aYtxYAHXRPWYe90fbypz5TMezHDiy0Jt8=+BPpQ@mail.gmail.com>
Date: Sat, 22 Nov 2025 09:31:06 +0800
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <726A237D-D1C1-49E5-99A0-AC5899897DA6@gmail.com>
References: 
 <CAHGQGwGYV+-abbKwdrM2UHUe-JYOFWmsrs6=QicyJO-j+-Widw@mail.gmail.com>
 <F7704488-8AE1-4744-AF8C-51181BAA631A@gmail.com>
 <CAHGQGwGYG09DZnx7VVFvr7EfpC12sf+36D6iCSSAFMj66RY2_Q@mail.gmail.com>
 <6E35E0EF-A842-408A-A264-DB6D428C2BD6@gmail.com>
 <CAHGQGwGnkM8aYtxYAHXRPWYe90fbypz5TMezHDiy0Jt8=+BPpQ@mail.gmail.com>
To: Fujii Masao <masao.fujii@gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/726A237D-D1C1-49E5-99A0-AC5899897DA6%40gmail.com>
Precedence: bulk


> On Nov 22, 2025, at 00:14, Fujii Masao <masao.fujii@gmail.com> wrote:
>=20
> On Fri, Nov 21, 2025 at 6:24=E2=80=AFPM Chao Li =
<li.evan.chao@gmail.com> wrote:
>> No, what I was thinking is that, we could combine the three set =
statement into one, like:
>>=20
>> ```
>> Set a =3D 1; set b =3D 2; set c =3D 3;
>> ```
>> So that sends a single statement to publisher server, that reduces =
round-trip from 3 times to one time.
>=20
> I see the point about combining the three SET commands to reduce round =
trips,
> but I think the current approach in the patch (i.e., issuing a =
separate
> SET command for each parameter) is sufficient. I still don't think
> the additional round trip during replication connection startup is
> a real concern. This approach is also consistent with what =
postgres_fdw
> and pg_dump already do.
>=20

No problem. I don=E2=80=99t have a strong option here. I just saw you =
mentioned overhead of round trip and thought to improve. But I agree =
that overhead is tiny, not a real concern.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/