From: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, David Fetter <david(at)fetter(dot)org>, Josh Kupershmidt <schmiddy(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net> |
Subject: | Re: pg_restore multiple --function options |
Date: | 2013-08-27 19:35:40 |
Message-ID: | 521CFF8C.3000802@vmware.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 27.08.2013 21:56, Tom Lane wrote:
> Heikki Linnakangas<hlinnakangas(at)vmware(dot)com> writes:
>> Would anyone object to backpatching that change to 9.3 ? The risk seems
>> very small, and it would be good to do the other options in the same
>> release as --table. It was an oversight to only do it for --table in 9.3.
>
>> Assuming no objections, I'll apply the attached patch to 9.3 and master
>> later tonight.
>
> I object, strongly. This is a feature addition, and has no business going
> in post-rc1, especially with no time for review.
Ok.
> As far as the function case goes, I'm not really thrilled about layering
> more functionality on that until we've come to some understanding about
> how to select from a group of overloaded functions. It may be that this
> is orthogonal to that issue ... or maybe not. I don't have any objection
> to fixing the non-function cases, as long as it's only in HEAD.
Huh, what's that issue?
As the code stands, you have to pass the argument types to the
--function flag, ie. --function="myfunc(integer)". It's annoyingly picky
about the spelling, as the it has to match exactly what pg_dump prints,
but it does handle selecting one function from a group of overloaded
ones. And that really is orthogonal to whether or not you can give
multiple --function arguments.
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Hannu Krosing | 2013-08-27 19:36:40 | Re: PL/pgSQL PERFORM with CTE |
Previous Message | Kevin Grittner | 2013-08-27 19:31:35 | Re: several questions about pg_dumpall, pg_start_backup, pg_basebackup and WAL |