pgsql: Rethink \crosstabview's argument parsing logic.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Rethink \crosstabview's argument parsing logic.
Date: 2016-04-15 02:54:42
Message-ID: E1aqtu6-0006DQ-2D@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Rethink \crosstabview's argument parsing logic.

\crosstabview interpreted its arguments in an unusual way, including
doing case-insensitive matching of unquoted column names, which is
surely not the right thing. Rip that out in favor of doing something
equivalent to the dequoting/case-folding rules used by other psql
commands. To keep it simple, change the syntax so that the optional
sort column is specified as a separate argument, instead of the
also-quite-unusual syntax that attached it to the colH argument with
a colon.

Also, rework the error messages to be closer to project style.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/6f0d6a507889d94a79c0d18577a0cb1ccc2b6815

Modified Files
--------------
doc/src/sgml/ref/psql-ref.sgml | 30 +--
src/bin/psql/command.c | 10 +-
src/bin/psql/common.c | 17 +-
src/bin/psql/crosstabview.c | 326 ++++++++--------------------
src/bin/psql/settings.h | 4 +-
src/test/regress/expected/psql_crosstab.out | 38 +++-
src/test/regress/sql/psql_crosstab.sql | 22 +-
7 files changed, 156 insertions(+), 291 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Fujii Masao 2016-04-15 03:16:56 Re: [COMMITTERS] pgsql: Add regression tests for multiple synchronous standbys.
Previous Message Andres Freund 2016-04-15 02:51:45 Re: [COMMITTERS] pgsql: Allow Pin/UnpinBuffer to operate in a lockfree manner.