Re: extensible options syntax for replication parser?

Robert Haas <robertmhaas@gmail.com>

From: Robert Haas <robertmhaas@gmail.com>

To: Fabien COELHO <coelho@cri.ensmp.fr>

Cc: "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>

Date: 2021-09-21T16:27:21Z

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 →

Flexible options for CREATE_REPLICATION_SLOT.
- 0266e98c6b86 15.0 landed
Flexible options for BASE_BACKUP.
- 0ba281cb4bf9 15.0 landed

Attachments

v6-0001-Flexible-options-for-BASE_BACKUP.patch (application/octet-stream) patch v6-0001
v6-0002-Flexible-options-for-CREATE_REPLICATION_SLOT.patch (application/octet-stream) patch v6-0002

On Fri, Sep 10, 2021 at 3:44 PM Robert Haas <robertmhaas@gmail.com> wrote:
> Last call for complaints about either the overall direction or the
> specific implementation choices...

A complaint showed up over at
http://postgr.es/m/979131631633278@mail.yandex.ru and pursuant to that
complaint I have made the new syntax for controlling the checkpoint
type look like CHECKPOINT { 'fast' | 'spread' } rather than just
having an option called FAST. It was suggested over there to also
rename WAIT to WAIT_WAL_ARCHIVED, but I don't like that for reasons
explained on that thread and so have not adopted that proposal.

Sergei also helpfully pointed out that I'd accidentally deleted a
version check in one place, so this version is also updated to not do
that.

-- 
Robert Haas
EDB: http://www.enterprisedb.com