pgsql: Show ignored constants as "$N" rather than "?" in pg_stat_statem

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Show ignored constants as "$N" rather than "?" in pg_stat_statem
Date: 2017-03-28 00:14:41
Message-ID: E1csemX-0000xF-Sc@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Show ignored constants as "$N" rather than "?" in pg_stat_statements.

The trouble with the original choice here is that "?" is a valid (and
indeed used) operator name, so that you could end up with ambiguous
statement texts like "SELECT ? ? ?". With this patch, you instead
see "SELECT $1 ? $2", which seems significantly more readable.
The numbers used for this purpose begin after the last actual $N parameter
in the particular query. The conflict with external parameters has its own
potential for confusion of course, but it was agreed to be an improvement
over the previous behavior.

Lukas Fittl

Discussion: https://postgr.es/m/CAP53PkxeaCuwYmF-A4J5z2-qk5fYFo5_NH3gpXGJJBxv1DMwEw@mail.gmail.com

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/a6f22e83562d8b78293229587cd3d9430d16d466

Modified Files
--------------
.../expected/pg_stat_statements.out | 113 ++++++++++++++-------
contrib/pg_stat_statements/pg_stat_statements.c | 40 ++++++--
.../pg_stat_statements/sql/pg_stat_statements.sql | 16 ++-
doc/src/sgml/pgstatstatements.sgml | 29 ++++--
4 files changed, 142 insertions(+), 56 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2017-03-28 00:54:47 Re: pgsql: Clean up Perl code according to perlcritic
Previous Message Tom Lane 2017-03-27 22:16:17 Re: [COMMITTERS] pgsql: Clean up Perl code according to perlcritic