Re: Add new protocol message to change GUCs for usage with future protocol-only GUCs

From: Jelte Fennema-Nio <me(at)jeltef(dot)nl>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Jacob Burroughs <jburroughs(at)instructure(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Dave Cramer <davecramer(at)gmail(dot)com>, "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Jeff Davis <pgsql(at)j-davis(dot)com>, Peter Eisentraut <peter(at)eisentraut(dot)org>
Subject: Re: Add new protocol message to change GUCs for usage with future protocol-only GUCs
Date: 2024-01-16 13:43:23
Message-ID: CAGECzQSxbEVyLGirxC4Jq3JfWHUUNOqG=TVH0UjZ7-B12zX=SQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 11 Jan 2024 at 17:20, Jelte Fennema-Nio <me(at)jeltef(dot)nl> wrote:
> So to be clear, if you consider 0008 the most controversial/risky part
> of this patchset (which it sounds like that's the case). I'd be fine
> with removing that for now.

I haven't removed 0008 yet, since I'd like some feedback first if that
makes sense. But I did add two new patches in the middle of the
patchset (which shift the later patch numbers by 2):

0007: Adds a new \parameterset meta-command to psql to be able to more
easily test the new ParameterSet message

0008: Shows warning in psql if the server is not supporting the newest
protocol version.

Attachment Content-Type Size
v7-0005-Bump-protocol-version-to-3.1.patch application/octet-stream 1014 bytes
v7-0002-libpq-Handle-NegotiateProtocolVersion-message-mor.patch application/octet-stream 8.8 KB
v7-0003-Prepare-server-code-for-addition-of-protocol-exte.patch application/octet-stream 2.0 KB
v7-0001-libpq-Remove-instance-of-hardcoded-protocol-versi.patch application/octet-stream 1.1 KB
v7-0004-libpq-Include-minor-version-number-in-PQprotocolV.patch application/octet-stream 3.3 KB
v7-0008-psql-Warn-about-unsupported-protocol-features.patch application/octet-stream 2.3 KB
v7-0006-Add-protocol-message-to-change-parameters.patch application/octet-stream 16.2 KB
v7-0007-psql-Add-parameterset-meta-command.patch application/octet-stream 8.1 KB
v7-0009-Add-GUC-contexts-for-protocol-extensions.patch application/octet-stream 5.9 KB
v7-0010-Add-_pq_.protocol_managed_params-protocol-extensi.patch application/octet-stream 11.8 KB
v7-0011-Add-tests-for-ParameterSet-and-_pq_.protocol_mana.patch application/octet-stream 24.5 KB
v7-0012-Add-_pq_.report_parameters-protocol-extension.patch application/octet-stream 23.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2024-01-16 13:55:12 Re: Add PQsendSyncMessage() to libpq
Previous Message David G. Johnston 2024-01-16 13:30:34 Re: New Window Function: ROW_NUMBER_DESC() OVER() ?