pgsql: psql: show proper row count in \x mode for zero-column output

Started by Bruce Momjianalmost 11 years ago4 messages
#1Bruce Momjian
bruce@momjian.us

psql: show proper row count in \x mode for zero-column output

Also, fix pager enable selection for such cases, and other cleanups for
zero-column output.

Report by Thom Brown

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/376a0c4547fe98c45476647596ce9c9b394f8415

Modified Files
--------------
src/bin/psql/print.c | 25 ++++++++++++++++++-------
1 file changed, 18 insertions(+), 7 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Peter Eisentraut
peter_e@gmx.net
In reply to: Bruce Momjian (#1)
Re: [COMMITTERS] pgsql: psql: show proper row count in \x mode for zero-column output

On 3/24/15 9:04 PM, Bruce Momjian wrote:

psql: show proper row count in \x mode for zero-column output

Also, fix pager enable selection for such cases, and other cleanups for
zero-column output.

Report by Thom Brown

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/376a0c4547fe98c45476647596ce9c9b394f8415

This change added an extra blank line to the output of a zero-row result.

Compare:

[9.4]
$ psql -X -d postgres -c 'select * from pg_class where false' -x
(No rows)
$

[9.5]
$ psql -X -d postgres -c 'select * from pg_class where false' -x
(0 rows)

$

Was that intentional?

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Jim Nasby
Jim.Nasby@BlueTreble.com
In reply to: Peter Eisentraut (#2)
Re: Re: [COMMITTERS] pgsql: psql: show proper row count in \x mode for zero-column output

On 6/25/15 9:42 AM, Peter Eisentraut wrote:

On 3/24/15 9:04 PM, Bruce Momjian wrote:

psql: show proper row count in \x mode for zero-column output

Also, fix pager enable selection for such cases, and other cleanups for
zero-column output.

Report by Thom Brown

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/376a0c4547fe98c45476647596ce9c9b394f8415

This change added an extra blank line to the output of a zero-row result.

Compare:

[9.4]
$ psql -X -d postgres -c 'select * from pg_class where false' -x
(No rows)
$

[9.5]
$ psql -X -d postgres -c 'select * from pg_class where false' -x
(0 rows)

$

Was that intentional?

That's consistent with what > 0 rows does, so it seems the correct thing
to do. Going from "(No rows)" to "(0 rows)" is going to break things
anyway, so I don't see a backwards compatibility issue here.
--
Jim Nasby, Data Architect, Blue Treble Consulting, Austin TX
Data in Trouble? Get it in Treble! http://BlueTreble.com

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#4Bruce Momjian
bruce@momjian.us
In reply to: Peter Eisentraut (#2)
Re: [COMMITTERS] pgsql: psql: show proper row count in \x mode for zero-column output

On Thu, Jun 25, 2015 at 10:42:54AM -0400, Peter Eisentraut wrote:

On 3/24/15 9:04 PM, Bruce Momjian wrote:

psql: show proper row count in \x mode for zero-column output

Also, fix pager enable selection for such cases, and other cleanups for
zero-column output.

Report by Thom Brown

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/376a0c4547fe98c45476647596ce9c9b394f8415

This change added an extra blank line to the output of a zero-row result.

Compare:

[9.4]
$ psql -X -d postgres -c 'select * from pg_class where false' -x
(No rows)
$

[9.5]
$ psql -X -d postgres -c 'select * from pg_class where false' -x
(0 rows)

$

Was that intentional?

I think so. ;-) We get a blank line for output of rows in -x:

$ psql -X -d postgres -c 'select * from pg_class where 1=1' -x
...
relispopulated | t
relreplident | n
relfrozenxid | 0
relminmxid | 0
relacl | {postgres=arwdDxt/postgres,=r/postgres}
reloptions |
-->
$

and for no rows without -x:

$ psql -X -d postgres -c 'select * from pg_class where 1=0'
relname | relnamespace | reltype | reloftype | relowner | relam | relfilenode | ...
---------+--------------+---------+-----------+----------+-------+-------------+-...
(0 rows)
-->
$

and when we return rows without -x:

$ psql -X -d postgres -c 'select * from pg_class where 1=1'
...
s,=r/postgres} |
user_mappings | 12086 | 12359 | 0 | ...
(311 rows)
-->
$

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ Everyone has their own god. +

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers