psql: add volatility to \df+

Started by Neil Conwayabout 19 years ago7 messagespatches
Jump to latest
#1Neil Conway
neilc@samurai.com

Attached is a patch that adds information about function volatility to
the output of psql's "\df+" slash command. I'll apply this to HEAD
tomorrow, barring any objections.

-Neil

Attachments:

volatility_psql_describe-1.patchtext/x-patch; charset=UTF-8; name=volatility_psql_describe-1.patchDownload+7-2
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Neil Conway (#1)
Re: psql: add volatility to \df+

Neil Conway <neilc@samurai.com> writes:

Attached is a patch that adds information about function volatility to
the output of psql's "\df+" slash command. I'll apply this to HEAD
tomorrow, barring any objections.

+1, but are there not any documentation changes to make?

regards, tom lane

#3Joshua D. Drake
jd@commandprompt.com
In reply to: Tom Lane (#2)
Re: psql: add volatility to \df+

Tom Lane wrote:

Neil Conway <neilc@samurai.com> writes:

Attached is a patch that adds information about function volatility to
the output of psql's "\df+" slash command. I'll apply this to HEAD
tomorrow, barring any objections.

+1, but are there not any documentation changes to make?

Well here is a question (just because I haven't seen it) is there a list
of functions and their volatility in the docs?

Joshua D. Drake

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

http://archives.postgresql.org

--

=== The PostgreSQL Company: Command Prompt, Inc. ===
Sales/Support: +1.503.667.4564 || 24x7/Emergency: +1.800.492.2240
Providing the most comprehensive PostgreSQL solutions since 1997
http://www.commandprompt.com/

Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate
PostgreSQL Replication: http://www.commandprompt.com/products/

#4Neil Conway
neilc@samurai.com
In reply to: Joshua D. Drake (#3)
Re: psql: add volatility to \df+

On Tue, 2007-26-06 at 20:29 -0700, Joshua D. Drake wrote:

Tom Lane wrote:

+1, but are there not any documentation changes to make?

Sure, I'll update the psql ref page.

Well here is a question (just because I haven't seen it) is there a list
of functions and their volatility in the docs?

Not relevant to this thread, but AFAICS there is no explicit list of the
volatilities of the builtin functions.

-Neil

#5Alvaro Herrera
alvherre@2ndquadrant.com
In reply to: Tom Lane (#2)
Re: psql: add volatility to \df+

Tom Lane wrote:

Neil Conway <neilc@samurai.com> writes:

Attached is a patch that adds information about function volatility to
the output of psql's "\df+" slash command. I'll apply this to HEAD
tomorrow, barring any objections.

+1, but are there not any documentation changes to make?

+1 as well but I'm wondering whether the output is too wide. Is it
possible to find some shorter string to convey the same meaning?
Ideally, without being a single character.

--
Alvaro Herrera http://www.amazon.com/gp/registry/CTMLCN8V17R4
"Saca el libro que tu religi�n considere como el indicado para encontrar la
oraci�n que traiga paz a tu alma. Luego rebootea el computador
y ve si funciona" (Carlos Ducl�s)

#6Neil Conway
neilc@samurai.com
In reply to: Alvaro Herrera (#5)
Re: psql: add volatility to \df+

On Wed, 2007-27-06 at 00:16 -0400, Alvaro Herrera wrote:

+1 as well but I'm wondering whether the output is too wide.

Well, the output of \df+ is already likely to be wider than 72
characters, so I'm not sure the patch would really make the situation
materially worse ("\df+ nextval" is 118 characters wide without the
patch, for example). You could also argue that if the user specifies
"+", presumably they are less worried about a concise output format.

Is it possible to find some shorter string to convey the same meaning?

Nothing obvious comes to mind, but I'm not opposed in principle. Any
suggestions?

-Neil

#7Neil Conway
neilc@samurai.com
In reply to: Neil Conway (#1)
Re: psql: add volatility to \df+

On Tue, 2007-26-06 at 15:10 -0700, Neil Conway wrote:

Attached is a patch that adds information about function volatility to
the output of psql's "\df+" slash command. I'll apply this to HEAD
tomorrow, barring any objections.

Applied.

-Neil