Horton

Re: Cleanup: remove unused fields from nodes

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Matthias van de Meent <boekewurm+postgres@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Etsuro Fujita <etsuro.fujita@gmail.com>, Amit Kapila <amit.kapila16@gmail.com>
Date: 2024-04-25T01:25:36Z
Lists: pgsql-hackers 
On Wed, Apr 24, 2024 at 08:31:57AM -0400, Tom Lane wrote:
> I was thinking about wording like
> 
>      * True if DEALLOCATE ALL.  This is redundant with "name == NULL",
>      * but we make it a separate field so that exactly this condition
>      * (and not the precise name) will be accounted for in query jumbling.

Fine by me.  I've just used that and applied a patch to doing so.
Thanks.
--
Michael

Commits

  1. Improve comment of DeallocateStmt->isall

  2. Remove some unnecessary fields from executor nodes.

  3. Add SQL/JSON query functions

  4. Add support for doing late row locking in FDWs.