Re: pgsql: Implement pg_wal_replay_wait() stored procedure

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>, Peter Eisentraut <peter(at)eisentraut(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgsql: Implement pg_wal_replay_wait() stored procedure
Date: 2024-11-04 14:19:26
Message-ID: 202411041419.qif3lqaisktw@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 2024-Nov-04, Alexander Korotkov wrote:

> On Mon, Nov 4, 2024 at 8:04 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:

> > Using output parameters in a procedure is something I did not recall.
> > Based on your point about not using a function due your argument based
> > on the snapshots, let's just use that and forget about the status
> > function entirely (please?).
>
> Please, check [1]. Usage of output parameters is a bit awkward too,
> because you need to pass some value in there. And more importantly,
> usage of output parameters also causes snapshot problem, as it causes
> another snapshot to be held.

I wonder if it would be better to go back to the original idea of using
special DDL syntax rather than a procedure. It seems we've been piling
up hacks to get around the behavior of procedures, and we seem to have
grown one more to handle repeatable read transactions.

It's looking to me like there's just too much cruft in the quest to
avoid having to reach consensus on new syntax. This might be a mistake.
Is it possible to backtrack on that decision?

--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"Linux transformó mi computadora, de una `máquina para hacer cosas',
en un aparato realmente entretenido, sobre el cual cada día aprendo
algo nuevo" (Jaime Salinas)

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Heikki Linnakangas 2024-11-04 14:21:50 pgsql: Move TRACE calls into WaitOnLock()
Previous Message Peter Geoghegan 2024-11-04 14:06:09 pgsql: Clarify nbtree parallel scan _bt_endpoint contract.

Browse pgsql-hackers by date

  From Date Subject
Next Message Tender Wang 2024-11-04 14:31:23 Fix a typo in the comment of gistdoinsert()
Previous Message Alexander Korotkov 2024-11-04 14:02:08 Re: pgsql: Implement pg_wal_replay_wait() stored procedure