| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com> |
| Cc: | Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Andrew Dunstan <andrew(at)dunslane(dot)net> |
| Subject: | Re: pgsql: libpq: Grease the protocol by default |
| Date: | 2026-02-24 02:08:56 |
| Message-ID: | 3864574.1771898936@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com> writes:
> On Mon, Feb 23, 2026 at 4:45 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Also: I was initially baffled why you thought this needs
>> back-patching, but I guess you have one eye on packagers like
>> Debian who think they can make older versions use newer libpq.so.
> Right.
Actually, that is going to be harder than you thought, because libpq
before v18 will spit up on connection option "max_protocol_version".
This patch will not work as-is for back-patching unless we care to
also back-patch the addition of that option, which I'd be inclined
to resist.
Fortunately, we long ago had the foresight to invent PQlibVersion,
so you could make addition of the extra option conditional on
PQlibVersion(conn) >= 180000 in branches before 18.
> Hmmm, looks like the -dump1.log output is actually from *before*
> pg_upgrade actually runs:
Yeah, I came to the same conclusion. I got a clean BF run using
your patch together with the attached patch for the BF client.
(In this patch, I did not worry about scenarios involving old
minor releases. If Andrew is excited about that case he can
extend the version-comparison logic.)
regards, tom lane
| Attachment | Content-Type | Size |
|---|---|---|
| cross-version.patch | text/x-diff | 931 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2026-02-24 03:36:27 | pgsql: fe_utils: Sprinkle some pg_malloc_object() and pg_malloc_array() |
| Previous Message | Tom Lane | 2026-02-24 01:17:23 | Re: pgsql: libpq: Grease the protocol by default |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David Rowley | 2026-02-24 02:23:17 | Re: More speedups for tuple deformation |
| Previous Message | Nitin Motiani | 2026-02-24 02:06:45 | Re: [PATCH] Support reading large objects with pg_read_all_data |