Re: Reducing output size of nodeToString

Matthias van de Meent <boekewurm+postgres@gmail.com>

From: Matthias van de Meent <boekewurm+postgres@gmail.com>

To:

Cc: Robert Haas <robertmhaas@gmail.com>, Peter Eisentraut <peter@eisentraut.org>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Andres Freund <andres@anarazel.de>, Michel Pelletier <pelletier.michel@gmail.com>

Date: 2024-02-12T19:32:56Z

Lists: pgsql-hackers

Attachments

v3-0002-pg_node_tree-Don-t-store-query-text-locations-in-.patch (application/octet-stream) patch v3-0002
v3-0004-gen_node_support.pl-Add-a-TypMod-type-for-signall.patch (application/octet-stream) patch v3-0004
v3-0005-nodeToString-omit-serializing-NULL-datums-in-Cons.patch (application/octet-stream) patch v3-0005
v3-0003-gen_node_support.pl-Mark-location-fields-as-type-.patch (application/octet-stream) patch v3-0003
v3-0006-nodeToString-Apply-RLE-on-Bitmapset-and-numeric-L.patch (application/octet-stream) patch v3-0006
v3-0007-gen_node_support.pl-Optimize-serialization-of-fie.patch (application/octet-stream) patch v3-0007
v3-0001-pg_node_tree-Omit-serialization-of-fields-with-de.patch (application/octet-stream) patch v3-0001

On Mon, 12 Feb 2024 at 19:03, Matthias van de Meent
<boekewurm+postgres@gmail.com> wrote:
> Attached is patchset v2, which contains the improvements from these patches:

Attached v3, which fixes an out-of-bounds read in pg_strtoken_next,
detected by asan, that was a likely cause of the problems in CFBot's
FreeBSD regression tests.

Kind regards,

Matthias van de Meent
Neon (https://neon.tech)

Commits

Do not output actual value of location fields in node serialization by default
- d20d8fbd3e4d 17.0 landed
gen_node_support.pl: Mark location fields as type alias ParseLoc
- 605721f819f5 17.0 landed