pg.ddx.io pgsql-committers@postgresql.org mailing list archive
help / color / mirror / Atom feedpglogparser - pglogparser: Initial checking to cvs
2+ messages / 2 participants
[nested] [flat]
* pglogparser - pglogparser: Initial checking to cvs
@ 2009-04-02 14:19 User Kpierno <kpierno@pgfoundry.org>
0 siblings, 0 replies; 2+ messages in thread
From: kpierno@pgfoundry.org @ 2009-04-02 14:19 UTC (permalink / raw)
To: pgsql-committers
Log Message:
-----------
Initial checking to cvs
Added Files:
-----------
pglogparser:
.project (r1.1)
(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/pglogparser/pglogparser/.project?rev=1.1&content-typ...)
logo.gif (r1.1)
(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/pglogparser/pglogparser/logo.gif?rev=1.1&content-typ...)
pglog_parser.pl (r1.1)
(http://cvs.pgfoundry.org/cgi-bin/cvsweb.cgi/pglogparser/pglogparser/pglog_parser.pl?rev=1.1&cont...)
^ permalink raw reply [nested|flat] 2+ messages in thread
* pgsql: Do not output actual value of location fields in node serializat
@ 2024-03-22 08:55 Peter Eisentraut <peter@eisentraut.org>
0 siblings, 0 replies; 2+ messages in thread
From: Peter Eisentraut @ 2024-03-22 08:55 UTC (permalink / raw)
To: pgsql-committers@lists.postgresql.org
Do not output actual value of location fields in node serialization by default
This changes nodeToString() to not output the actual value of location
fields in nodes, but instead it writes -1. This mirrors the fact that
stringToNode() also does not read location field values but always
stores -1.
For most uses of nodeToString(), which is to store nodes in catalog
fields, this is more useful. We don't store original query texts in
catalogs, so any lingering query location values are not meaningful.
For debugging purposes, there is a new nodeToStringWithLocations(),
which mirrors the existing stringToNodeWithLocations(). This is used
for WRITE_READ_PARSE_PLAN_TREES and nodes/print.c functions, which
covers all the debugging uses.
Reviewed-by: Matthias van de Meent <boekewurm+postgres@gmail.com>
Discussion: https://www.postgresql.org/message-id/flat/CAEze2WgrCiR3JZmWyB0YTc8HV7ewRdx13j0CqD6mVkYAW+SFGQ@mail....
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/d20d8fbd3e4d327dd0778a02d5661b51f4f6423a
Modified Files
--------------
src/backend/nodes/outfuncs.c | 37 ++++++++++++++++++++++++++++++++++---
src/backend/nodes/print.c | 6 +++---
src/backend/tcop/postgres.c | 6 +++---
src/include/nodes/nodes.h | 1 +
4 files changed, 41 insertions(+), 9 deletions(-)
^ permalink raw reply [nested|flat] 2+ messages in thread
end of thread, other threads:[~2024-03-22 08:55 UTC | newest]
Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2009-04-02 14:19 pglogparser - pglogparser: Initial checking to cvs User Kpierno <kpierno@pgfoundry.org>
2024-03-22 08:55 pgsql: Do not output actual value of location fields in node serializat Peter Eisentraut <peter@eisentraut.org>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox