Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c
Дата
Msg-id CA+TgmoYXayXNqohhXvFHVPKmaNa9EwZYN4A5H-XX-5EZW0QS2w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c
Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c
Список pgsql-hackers
On Thu, Aug 4, 2011 at 2:04 PM, Robert Haas <robertmhaas@gmail.com> wrote:
> On Thu, Aug 4, 2011 at 1:53 PM, Phil Sorber <phil@omniti.com> wrote:
>> Ok, here is the patch that just moves the ALTER/SET pieces to the end.
>> Can we get this included in the next commit fest?
>
> Yep, just make yourself an account and add it.

Unfortunately, it doesn't look like anyone ever replied to this
thread, but Tom posted some thoughts on another thread that seem to me
to be a serious problem for this patch:

http://archives.postgresql.org/message-id/13764.1315094292@sss.pgh.pa.us

I don't see any easy way around that problem, so I'm going to mark
this patch Returned with Feedback for now.  It strikes me as craziness
to try to guess which settings we should restore at the beginning and
which at the end, so I think we need a better idea.  I don't really
understand why it's not OK to just have pg_dump issue RESET ROLE at
appropriate points in the process; that seems like it would be
sufficient and not particularly ugly.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: [v9.2] Fix Leaky View Problem
Следующее
От: David Fetter
Дата:
Сообщение: Re: Extend file_fdw wrapper