pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Bruce Momjian <bruce@momjian.us>
To: bzm@2bz.de
To: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #17957: psql-16beta1 \df+ does not show the function body
Date: Thu, 1 Jun 2023 15:23:12 -0400
Message-ID: <ZHjwIHz/26HITdUC@momjian.us> (raw)
In-Reply-To: <17957-e1db8b1c15af444a@postgresql.org>
References: <17957-e1db8b1c15af444a@postgresql.org>

On Thu, Jun  1, 2023 at 05:57:19PM +0000, PG Bug reporting form wrote:
> The following bug has been logged on the website:
> 
> Bug reference:      17957
> Logged by:          Boris Z
> Email address:      bzm@2bz.de
> PostgreSQL version: 16beta1
> Operating system:   OSX, Linux
> Description:        
> 
> Hi,
> 
> The release notes do not mention \df+. 
> https://www.postgresql.org/docs/16/release-16.html#RELEASE-16-PSQL
> 
> for testing I used this function, but any user function will do.
> 
> CREATE OR REPLACE FUNCTION test_function()
> RETURNS bigint
> AS $$
>     SELECT 1;
> $$ LANGUAGE sql
> STABLE
> PARALLEL SAFE;
> 
> When using psql (16beta1)
> =# \df+ test_function
> -[ RECORD 1 ]-------+--------------
> Schema              | public
> Name                | test_function
> Result data type    | bigint
> Argument data types |
> Type                | func
> Volatility          | stable
> Parallel            | safe
> Owner               | bz
> Security            | invoker
> Access privileges   |
> Language            | sql
> Internal name       |
> Description         |
> 
> using psql-15.3 it works as expected
> =# \df+ test_function
> -[ RECORD 1 ]-------+--------------
> Schema              | public
> Name                | test_function
> Result data type    | bigint
> Argument data types |
> Type                | func
> Volatility          | stable
> Parallel            | safe
> Owner               | bz
> Security            | invoker
> Access privileges   |
> Language            | sql
> Source code         |              +
>                     |     SELECT 1;+
>                     |
> Description         |

We had this commit in PG 16:

	Author: Tom Lane <tgl@sss.pgh.pa.us>
	2023-03-02 [3dfae91f7] Show "internal name" not "source code" in psql's \df+ co
	
	    Show "internal name" not "source code" in psql's \df+ command.
	
	    Our previous habit of showing the full function body is really
	    pretty unfriendly for tabular viewing of functions, and now that
	    we have \sf and \ef commands there seems no good reason why \df+
	    has to do it.  It still seems to make sense to show prosrc for
	    internal and C-language functions, since in those cases prosrc
	    is just the C function name; but then let's rename the column to
	    "Internal name" which is a more accurate descriptor.
	
	    Isaac Morland
	
	    Discussion: https://postgr.es/m/CAMsGm5eqKc6J1=Lwn=ZONG=6ZDYWRQ4cgZQLqMuZGB1aVt_JBg@mail.gmail.com

I didn't think anyone would mind the change so I didn't document it in
the release notes --- should I?

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.





view thread (4+ messages)  latest in thread

Message-ID: <ZHjwIHz/26HITdUC@momjian.us>
Permalink:  ../ZHjwIHz%2F26HITdUC@momjian.us/
Also on:    postgresql.org/message-id/ZHjwIHz/26HITdUC@momjian.us

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: bruce@momjian.us, bzm@2bz.de, pgsql-bugs@lists.postgresql.org
  Subject: Re: BUG #17957: psql-16beta1 \df+ does not show the function body
  In-Reply-To: <ZHjwIHz/26HITdUC@momjian.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox