|
Home > Archive > PostgreSQL Hacks > October 2005 > expanded \df+ display broken in beta4
You are viewing an archived Text-only version of the thread.
To view this thread in it's original format and/or if you want to reply to
this thread please [click here]
| Author |
expanded \df+ display broken in beta4
|
|
| Robert Treat 2005-10-27, 8:14 am |
| not sure exactly when this was changed, but expanded display of \df+
output is broken in beta4. compare:
& #91;postgres@localho
st data]$ /usr/local/pgsql-8.1.x/bin/psql -p 5481
template1
Welcome to psql 8.1beta4, the PostgreSQL interactive terminal.
Type: \copyright for distribution terms
\h for help with SQL commands
\? for help with psql commands
\g or terminate with semicolon to execute query
\q to quit
template1=# \df+ area
List of functions
Schema | Name | Result data type | Argument data types | Owner |
Language | Source code | Description
------------+------+------------------+---------------------+----------+----------+-------------+-----------------------
pg_catalog | area | double precision | box | postgres |
internal | box_area | box area
pg_catalog | area | double precision | circle | postgres |
internal | circle_area | area of circle
pg_catalog | area | double precision | path | postgres |
internal | path_area | area of a closed path
(3 rows)
template1=# \x
Expanded display is on.
template1=# \df+ area
List of functions
Schema | Name | Result data type | Argument data types | Owner |
Language | Source code | Description
------------+------+------------------+---------------------+----------+----------+-------------+-----------------------
pg_catalog | area | double precision | box | postgres |
internal | box_area | box area
pg_catalog | area | double precision | circle | postgres |
internal | circle_area | area of circle
pg_catalog | area | double precision | path | postgres |
internal | path_area | area of a closed path
& #91;postgres@localho
st data]$ /usr/local/pgsql-8.0.x/bin/psql -p 5481
template1
Welcome to psql 8.0.4, the PostgreSQL interactive terminal.
Type: \copyright for distribution terms
\h for help with SQL commands
\? for help with psql commands
\g or terminate with semicolon to execute query
\q to quit
template1=# \df+ area
List of functions
Schema | Name | Result data type | Argument data types | Owner |
Language | Source code | Description
------------+------+------------------+---------------------+----------+----------+-------------+-----------------------
pg_catalog | area | double precision | box | postgres |
internal | box_area | box area
pg_catalog | area | double precision | circle | postgres |
internal | circle_area | area of circle
pg_catalog | area | double precision | path | postgres |
internal | path_area | area of a closed path
(3 rows)
template1=# \x
Expanded display is on.
template1=# \df+ area
List of functions
-[ RECORD 1 ]-------+----------------------
Schema | pg_catalog
Name | area
Result data type | double precision
Argument data types | box
Owner | postgres
Language | internal
Source code | box_area
Description | box area
-[ RECORD 2 ]-------+----------------------
Schema | pg_catalog
Name | area
Result data type | double precision
Argument data types | circle
Owner | postgres
Language | internal
Source code | circle_area
Description | area of circle
-[ RECORD 3 ]-------+----------------------
Schema | pg_catalog
Name | area
Result data type | double precision
Argument data types | path
Owner | postgres
Language | internal
Source code | path_area
Description | area of a closed path
template1=#
certainly the second is much more legible... this is on... FC3 in case
it matters.
Robert Treat
--
Build A Brighter Lamp :: Linux apache {middleware} PostgreSQL
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster
| |
|
|
| Bruce Momjian 2005-10-27, 8:14 am |
|
Good point. We modified 8.1 so backslash commands do not honor \x
because things like \d look silly in \x, but \df+ looks better with \x,
no question.
Ideally I think \x should allow three modes, on, off, and auto, with
auto doing \x if the row output is wider than the screen. If we had
this, backslash commands could be auto, or we can set all queries to
auto by default.
Added to TODO:
o Add auto-expanded mode so expanded output is used if the row
length is wider than the screen width.
Consider using auto-expanded mode for backslash commands like \df+.
---------------------------------------------------------------------------
Robert Treat wrote:
> not sure exactly when this was changed, but expanded display of \df+
> output is broken in beta4. compare:
>
>
> & #91;postgres@localho
st data]$ /usr/local/pgsql-8.1.x/bin/psql -p 5481
> template1
> Welcome to psql 8.1beta4, the PostgreSQL interactive terminal.
>
> Type: \copyright for distribution terms
> \h for help with SQL commands
> \? for help with psql commands
> \g or terminate with semicolon to execute query
> \q to quit
>
> template1=# \df+ area
> List of functions
> Schema | Name | Result data type | Argument data types | Owner |
> Language | Source code | Description
> ------------+------+------------------+---------------------+----------+----------+-------------+-----------------------
> pg_catalog | area | double precision | box | postgres |
> internal | box_area | box area
> pg_catalog | area | double precision | circle | postgres |
> internal | circle_area | area of circle
> pg_catalog | area | double precision | path | postgres |
> internal | path_area | area of a closed path
> (3 rows)
>
> template1=# \x
> Expanded display is on.
> template1=# \df+ area
> List of functions
> Schema | Name | Result data type | Argument data types | Owner |
> Language | Source code | Description
> ------------+------+------------------+---------------------+----------+----------+-------------+-----------------------
> pg_catalog | area | double precision | box | postgres |
> internal | box_area | box area
> pg_catalog | area | double precision | circle | postgres |
> internal | circle_area | area of circle
> pg_catalog | area | double precision | path | postgres |
> internal | path_area | area of a closed path
>
>
> & #91;postgres@localho
st data]$ /usr/local/pgsql-8.0.x/bin/psql -p 5481
> template1
> Welcome to psql 8.0.4, the PostgreSQL interactive terminal.
>
> Type: \copyright for distribution terms
> \h for help with SQL commands
> \? for help with psql commands
> \g or terminate with semicolon to execute query
> \q to quit
>
> template1=# \df+ area
> List of functions
> Schema | Name | Result data type | Argument data types | Owner |
> Language | Source code | Description
> ------------+------+------------------+---------------------+----------+----------+-------------+-----------------------
> pg_catalog | area | double precision | box | postgres |
> internal | box_area | box area
> pg_catalog | area | double precision | circle | postgres |
> internal | circle_area | area of circle
> pg_catalog | area | double precision | path | postgres |
> internal | path_area | area of a closed path
> (3 rows)
>
> template1=# \x
> Expanded display is on.
> template1=# \df+ area
> List of functions
> -[ RECORD 1 ]-------+----------------------
> Schema | pg_catalog
> Name | area
> Result data type | double precision
> Argument data types | box
> Owner | postgres
> Language | internal
> Source code | box_area
> Description | box area
> -[ RECORD 2 ]-------+----------------------
> Schema | pg_catalog
> Name | area
> Result data type | double precision
> Argument data types | circle
> Owner | postgres
> Language | internal
> Source code | circle_area
> Description | area of circle
> -[ RECORD 3 ]-------+----------------------
> Schema | pg_catalog
> Name | area
> Result data type | double precision
> Argument data types | path
> Owner | postgres
> Language | internal
> Source code | path_area
> Description | area of a closed path
>
> template1=#
>
>
> certainly the second is much more legible... this is on... FC3 in case
> it matters.
>
>
> Robert Treat
> --
> Build A Brighter Lamp :: Linux apache {middleware} PostgreSQL
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 2: Don't 'kill -9' the postmaster
>
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster
| |
| Martijn van Oosterhout 2005-10-27, 8:14 am |
| On Tue, Oct 25, 2005 at 02:51:04PM -0400, Bruce Momjian wrote:
>
> Good point. We modified 8.1 so backslash commands do not honor \x
> because things like \d look silly in \x, but \df+ looks better with \x,
> no question.
>
> Ideally I think \x should allow three modes, on, off, and auto, with
> auto doing \x if the row output is wider than the screen. If we had
> this, backslash commands could be auto, or we can set all queries to
> auto by default.
I have a patch on my local system that fixes the splattering across the
screen you currently get with multiline function definitions. Maybe
once that has been fixed we should revisit this.
Have a nice day,
--
Martijn van Oosterhout <kleptog@svana.org> http://svana.org/kleptog/
> Patent. n. Genius is 5% inspiration and 95% perspiration. A patent is a
> tool for doing 5% of the work and then sitting around waiting for someone
> else to do the other 95% so you can sue them.
| |
| Bruce Momjian 2005-10-27, 8:14 am |
| Martijn van Oosterhout wrote:
-- Start of PGP signed section.
> On Tue, Oct 25, 2005 at 02:51:04PM -0400, Bruce Momjian wrote:
>
> I have a patch on my local system that fixes the splattering across the
> screen you currently get with multiline function definitions. Maybe
> once that has been fixed we should revisit this.
Yes, but \df+ is not multi-line per column.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to majordomo@postgresql
.org so that your
message can get through to the mailing list cleanly
| |
| Robert Treat 2005-10-27, 8:14 am |
| On Tue, 2005-10-25 at 14:42, Michael Fuhr wrote:
> On Tue, Oct 25, 2005 at 02:27:50PM -0400, Robert Treat wrote:
>
> http://archives.postgresql.org/pgsq...06/msg00423.php
> http://archives.postgresql.org/pgsq...06/msg00149.php
>
grr... I thought I remembered Chris KL mentioning something about that
after I sent my email... and I see now that I actually did reply to that
thread saying I liked the \x \df+ output...
Robert Treat
--
Build A Brighter Lamp :: Linux apache {middleware} PostgreSQL
---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to majordomo@postgresql
.org so that your
message can get through to the mailing list cleanly
| |
| Robert Treat 2005-10-27, 8:14 am |
| On Tue, 2005-10-25 at 14:51, Bruce Momjian wrote:
>
> Good point. We modified 8.1 so backslash commands do not honor \x
> because things like \d look silly in \x, but \df+ looks better with \x,
> no question.
>
> Ideally I think \x should allow three modes, on, off, and auto, with
> auto doing \x if the row output is wider than the screen. If we had
> this, backslash commands could be auto, or we can set all queries to
> auto by default.
>
ISTM even a GUC to enable/disable would have been better scheme than
what we have now; we are basically leaving no options for those who
found the old behavior useful, while what we had before would at least
let people switch back and forth.
Robert Treat
--
Build A Brighter Lamp :: Linux apache {middleware} PostgreSQL
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster
| |
| Michael Paesold 2005-10-27, 8:14 am |
| Robert Treat wrote:
> On Tue, 2005-10-25 at 14:51, Bruce Momjian wrote:
>
> ISTM even a GUC to enable/disable would have been better scheme than
> what we have now; we are basically leaving no options for those who
> found the old behavior useful, while what we had before would at least
> let people switch back and forth.
I think Robert is right here and the new behaviour is a step backwards.
Best Regards,
Michael Paesold
---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
choose an index scan if your joining column's datatypes do not
match
| |
| Tom Lane 2005-10-27, 8:14 am |
| "Michael Paesold" <mpaesold@gmx.at> writes:
> Robert Treat wrote:
[color=darkred]
> I think Robert is right here and the new behaviour is a step backwards.
Should we revert the patch for the time being, and take another go at it
in 8.2?
Is it practical to have the old behavior for \df (and anything else with
particularly wide output) while still forcing \x off for \d?
regards, tom lane
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster
| |
| Michael Paesold 2005-10-27, 8:14 am |
| Tom Lane wrote:
> "Michael Paesold" <mpaesold@gmx.at> writes:
>
>
>
>
>
> Should we revert the patch for the time being, and take another go at it
> in 8.2?
As the last option, if there is no quick fix, I'd say yes. Better than
making an incomplete change now and revert that later to get a better
solution.
> Is it practical to have the old behavior for \df (and anything else with
> particularly wide output) while still forcing \x off for \d?
I think there are quite some ways to handle the problem:
1) \x to get the new behaviour, \xx or \x+ to get the new one.
2) Have \x+ or something automatically switch based on screen width.
Other suggestions...
Well, unfortunately, I guess now is not the time to discuss this for 8.1.
Best Regards,
Michael Paesold
---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?
http://www.postgresql.org/docs/faq
| |
| Bruce Momjian 2005-10-27, 8:14 am |
| Michael Paesold wrote:
> Tom Lane wrote:
>
> As the last option, if there is no quick fix, I'd say yes. Better than
> making an incomplete change now and revert that later to get a better
> solution.
>
>
> I think there are quite some ways to handle the problem:
> 1) \x to get the new behaviour, \xx or \x+ to get the new one.
>
> 2) Have \x+ or something automatically switch based on screen width.
>
> Other suggestions...
> Well, unfortunately, I guess now is not the time to discuss this for 8.1.
One idea is to hack \d not to honor \x, and let the others honor it.
That would probably hit most of the cases people will use in 8.1.
In fact, \d is pretty special because it is more of a group of outputs,
unlike \df, which is a single table output.
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster
| |
| Michael Paesold 2005-10-27, 8:14 am |
| Bruce Momjian wrote:
> Michael Paesold wrote:
>
[color=darkred]
> One idea is to hack \d not to honor \x, and let the others honor it.
> That would probably hit most of the cases people will use in 8.1.
>
> In fact, \d is pretty special because it is more of a group of outputs,
> unlike \df, which is a single table output.
+1 from me. That seems like a workable compromise and should probably
meet the needs of the author of the patch to change the \x behavior.
Best Regards,
Michael Paesold
---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?
http://www.postgresql.org/docs/faq
|
|
|
|
|