Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c - Mailing list pgsql-hackers

From Robert Haas
Subject Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c
Date
Msg-id CA+TgmoYXayXNqohhXvFHVPKmaNa9EwZYN4A5H-XX-5EZW0QS2w@mail.gmail.com
Whole thread Raw
In response to Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c
Re: patch: move dumpUserConfig call in dumpRoles function of pg_dumpall.c
List 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 by date:

Previous
From: Robert Haas
Date:
Subject: Re: [v9.2] Fix Leaky View Problem
Next
From: David Fetter
Date:
Subject: Re: Extend file_fdw wrapper