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

Jelte Fennema-Nio <me@jeltef.nl>

From: Jelte Fennema-Nio <me@jeltef.nl>

To: Peter Smith <smithpb2250@gmail.com>

Cc: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Jacob Burroughs <jburroughs@instructure.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Dave Cramer <davecramer@gmail.com>, "Andrey M. Borodin" <x4mmm@yandex-team.ru>, Pavel Stehule <pavel.stehule@gmail.com>, Jeff Davis <pgsql@j-davis.com>, Peter Eisentraut <peter@eisentraut.org>

Date: 2024-01-22T09:19:57Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

libpq: Add min/max_protocol_version connection options
- 285613c60a7a 18.0 landed
libpq: Handle NegotiateProtocolVersion message differently
- 5070349102af 18.0 landed
Add PQfullProtocolVersion() to surface the precise protocol version.
- cdb6b0fdb0b2 18.0 landed
Do not hardcode PG_PROTOCOL_LATEST in NegotiateProtocolVersion
- 516b87502dc1 18.0 landed
libpq: Handle NegotiateProtocolVersion message
- bbf9c282ce92 16.0 cited
Provide for forward compatibility with future minor protocol versions.
- ae65f6066dc3 11.0 cited

Attachments

v8-0005-libpq-Include-minor-version-number-in-PQprotocolV.patch (application/octet-stream) patch v8-0005
v8-0001-libpq-Remove-instance-of-hardcoded-protocol-versi.patch (application/octet-stream) patch v8-0001
v8-0003-libpq-Add-PQunsupportedProtocolExtensionParameter.patch (application/octet-stream) patch v8-0003
v8-0004-Prepare-server-code-for-addition-of-protocol-exte.patch (application/octet-stream) patch v8-0004
v8-0002-libpq-Handle-NegotiateProtocolVersion-message-mor.patch (application/octet-stream) patch v8-0002
v8-0006-Bump-protocol-version-to-3.1.patch (application/octet-stream) patch v8-0006
v8-0008-psql-Add-parameterset-meta-command.patch (application/octet-stream) patch v8-0008
v8-0007-Add-protocol-message-to-change-parameters.patch (application/octet-stream) patch v8-0007
v8-0009-psql-Warn-about-unsupported-protocol-features.patch (application/octet-stream) patch v8-0009
v8-0010-Add-GUC-contexts-for-protocol-extensions.patch (application/octet-stream) patch v8-0010
v8-0011-Add-_pq_.protocol_managed_params-protocol-extensi.patch (application/octet-stream) patch v8-0011
v8-0012-Add-tests-for-ParameterSet-and-_pq_.protocol_mana.patch (application/octet-stream) patch v8-0012
v8-0013-Add-_pq_.report_parameters-protocol-extension.patch (application/octet-stream) patch v8-0013

On Mon, 22 Jan 2024 at 04:31, Peter Smith <smithpb2250@gmail.com> wrote:
> Hi, This patch has a CF status of "Needs Review", but it seems like
> there were some CFbot test failures last time it was run [1]. Please
> have a look and post an updated version if necessary.

Ah yes, I had noticed that a while back and fixed the problem in v7 of
my patchset but it seems a rebase conflict has caused the cfbot not to
run tests on that version at all. Attached is a rebased version
together with following small changes:

- Rename PQunsupportedProtocolExtensions to
PQunsupportedProtocolExtensionParameters

- Add PQunsupportedProtocolExtensionParameters in its own patch