Re: [PATCH] "\ef " in psql - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] "\ef " in psql
Date
Msg-id 15645.1216826361@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] "\ef " in psql  ("Marko Kreen" <markokr@gmail.com>)
Responses Re: [PATCH] "\ef " in psql  ("Asko Oja" <ascoja@gmail.com>)
List pgsql-hackers
"Marko Kreen" <markokr@gmail.com> writes:
>> [ re pg_get_functiondef ]

> Please make it use full qualified names (schema.name) for both
> function name and result types.  Current search_path juggling
> the pg_dump does is major PITA.

Qualifying the function name seems like a good idea, but I'd advise
against tinkering with the datatype references.  It'll be hard to
do correctly and it will make things very substantially uglier.
Do you really want to show, eg, "pg_catalog.int4" rather than "integer"?

If you leave the backend code do what it wants to do here, the only
way that there would be a problem is if someone changed their
search_path in between pg_get_functiondef and trying to re-load the
function definition.  Which certainly ain't gonna happen for \ef,
and it seems a bit implausible for any other use-case either.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Zdenek Kotala
Date:
Subject: Re: pltcl_*mod commands are broken on Solaris 10
Next
From: Tom Lane
Date:
Subject: Re: pltcl_*mod commands are broken on Solaris 10