pgsql: Improve pretty printing of viewdefs.

Started by Andrew Dunstanalmost 14 years ago4 messages
#1Andrew Dunstan
andrew@dunslane.net

Improve pretty printing of viewdefs.

Some line feeds are added to target lists and from lists to make
them more readable. By default they wrap at 80 columns if possible,
but the wrap column is also selectable - if 0 it wraps after every
item.

Andrew Dunstan, reviewed by Hitoshi Harada.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/2f582f76b1945929ff07116cd4639747ce9bb8a1

Modified Files
--------------
doc/src/sgml/func.sgml | 12 ++-
src/backend/utils/adt/ruleutils.c | 137 +++++++++++++++++++++++++++-
src/include/catalog/catversion.h | 2 +-
src/include/catalog/pg_proc.h | 2 +
src/include/utils/builtins.h | 1 +
src/test/regress/expected/polymorphism.out | 4 +-
src/test/regress/expected/rules.out | 33 +++++++
src/test/regress/expected/with.out | 25 +++---
src/test/regress/sql/rules.sql | 6 +
9 files changed, 204 insertions(+), 18 deletions(-)

#2Pavel Stehule
pavel.stehule@gmail.com
In reply to: Andrew Dunstan (#1)
Re: pgsql: Improve pretty printing of viewdefs.

Hello

nice

should be this functionality used for query too?

some like

pg_pretty_query('SELECT ... ', 80)

when we have this functionality.

Regards

Pavel

2012/2/19 Andrew Dunstan <andrew@dunslane.net>:

Show quoted text

Improve pretty printing of viewdefs.

Some line feeds are added to target lists and from lists to make
them more readable. By default they wrap at 80 columns if possible,
but the wrap column is also selectable - if 0 it wraps after every
item.

Andrew Dunstan, reviewed by Hitoshi Harada.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/2f582f76b1945929ff07116cd4639747ce9bb8a1

Modified Files
--------------
doc/src/sgml/func.sgml                     |   12 ++-
src/backend/utils/adt/ruleutils.c          |  137 +++++++++++++++++++++++++++-
src/include/catalog/catversion.h           |    2 +-
src/include/catalog/pg_proc.h              |    2 +
src/include/utils/builtins.h               |    1 +
src/test/regress/expected/polymorphism.out |    4 +-
src/test/regress/expected/rules.out        |   33 +++++++
src/test/regress/expected/with.out         |   25 +++---
src/test/regress/sql/rules.sql             |    6 +
9 files changed, 204 insertions(+), 18 deletions(-)

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

#3Andrew Dunstan
andrew@dunslane.net
In reply to: Pavel Stehule (#2)
Re: pgsql: Improve pretty printing of viewdefs.

[redirecting to -hackers]

On 02/19/2012 12:04 PM, Pavel Stehule wrote:

Hello

nice

should be this functionality used for query too?

some like

pg_pretty_query('SELECT ... ', 80)

when we have this functionality.

It would probably be possible to leverage some of this for that, but
it's certainly not part of the present piece of work. All the logic is
there in get_query_def() and friends. There would need to be a wrapper
that called the parser to get a query object from the input string and
then called get_query_def() to get back the reformatted output, and
there are probably any number of wrinkles I haven't thought of.

cheers

andrew

#4Andrew Dunstan
adunstan@postgresql.org
In reply to: Andrew Dunstan (#3)
Re: [COMMITTERS] pgsql: Improve pretty printing of viewdefs.

On 02/19/2012 04:18 PM, Andrew Dunstan wrote:

[redirecting to -hackers]

Arghh, this time redirecting ...

Show quoted text

On 02/19/2012 12:04 PM, Pavel Stehule wrote:

Hello

nice

should be this functionality used for query too?

some like

pg_pretty_query('SELECT ... ', 80)

when we have this functionality.

It would probably be possible to leverage some of this for that, but
it's certainly not part of the present piece of work. All the logic is
there in get_query_def() and friends. There would need to be a wrapper
that called the parser to get a query object from the input string and
then called get_query_def() to get back the reformatted output, and
there are probably any number of wrinkles I haven't thought of.

cheers

andrew