Re: [HACKERS] pg_dump roles support

2008-11-13 Thread Benedek László
On 2008-11-08 09:25, Benedek László wrote: Does this work if the role name contains a ' ? Right, this one fails with ' in the role name. An update coming soon closing this issue. Here is an updated patch, which deals with 's in the rolename. Please review. doc/src/sgml/r

Re: [HACKERS] pg_dump roles support

2008-11-08 Thread Benedek László
Hi, Thank you for your review. On 2008-11-07 21:20, Alvaro Herrera wrote: The patch contains the following things: - pg_dump and pg_dumpall accepts the --role=rolename parameter, and sends a SET ROLE command on their connections Minor comment -- I think you need to quote the role name

Re: [HACKERS] pg_dump roles support [Review]

2008-11-06 Thread Benedek László
Hi, Thanks for your review. I created an updated patch according to your notices. 1 - Patch does not apply cleanly on latest git repository, although there is no hunk failed but there are some hunk succeeded messages. Rebased to the current HEAD. 2- Patch contains unnecessary spaces and tab

Re: [HACKERS] pg_dump roles support

2008-10-10 Thread Benedek László
Hello All, in my last mail http://www.mail-archive.com/pgsql-hackers@postgresql.org/msg114835.html I have sent you a patch of pg_dump and pg_dumpall --role option support. Nobody replied or commented jet so now I send it again. The attached patch is the same as the last one, except it is agai

Re: [HACKERS] pg_dump roles support

2008-09-24 Thread Benedek László
Hello, I modified my previous patch supporting the --role option in pg_dump and pg_dumpall. The attached patch contains the following things: - pg_dump and pg_dumpall accepts the --role=rolename parameter, and sends a SET ROLE command on their connections - pg_dumpall passes this option to t

Re: [HACKERS] pg_dump roles support

2008-09-04 Thread Benedek László
Tom Lane wrote: Some other review nitpicking: Thank you for your review. I really need all suggestions, since I never posted any patch to the community before. The next patch will emit the SET ROLE command in the generated dump, as you and Stephen said. This will fit in my workflow too, si

Re: [HACKERS] pg_dump roles support

2008-08-27 Thread Benedek László
Hello, daveg wrote: I created a patch to set the role to a specified name just after the db connection. I was going to do this, but you have beat me to it. You will want to update the sgml documentation, and pg_dumpall as well. -dg Ok, here is the next one. pg_dumpall now just pass

Re: [HACKERS] pg_dump roles support

2008-08-26 Thread Benedek László
Hello, Stephen Frost wrote: > As I discuss above, it'd be really nice have a --role or similar option > to ask pg_dump to set role to a particular user before dumping the > database. I created a patch to set the role to a specified name just after the db connection. Please review it for possibl