From: Jaime Casanova <systemguards@gmail.com>
To: Alvaro Herrera <alvherre@commandprompt.com>
Cc: pgsql-patches <pgsql-patches@postgresql.org>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [PATCHES] Extending grant insert on tables to sequences
Date: Tue, 8 Jul 2008 11:10:45 -0500
Message-ID: <c2d9e70e0807080910m7cb65cd0i3ecb449ca8fed17d@mail.gmail.com> (raw)
In-Reply-To: <20080708133244.GB4095@alvh.no-ip.org>
References: <c2d9e70e0805221118x1411e00crd7d4aae7e6e4ea3d@mail.gmail.com>
<c2d9e70e0805232156p20dbe2b9j2a88d35f951d625e@mail.gmail.com>
<20080708133244.GB4095@alvh.no-ip.org>
On 7/8/08, Alvaro Herrera <alvherre@commandprompt.com> wrote:
> Jaime Casanova escribió:
> > On Thu, May 22, 2008 at 1:18 PM, Jaime Casanova <systemguards@gmail.com> wrote:
> > > Hi,
> > >
> > > The idea of this patch is to avoid the need to make explicit grants on
> > > sequences owned by tables.
> >
> > I've noted that the patch i attached is an older version that doesn't
> > compile because of a typo...
> > Re-attaching right patch and fix documentation to indicate the new behaviour...
>
> I had a look at this patch and it looks good. The only thing that's not
> clear to me is whether we have agreed we want this to be the default
> behavior?
>
mmm... i don't remember from where i took the equivalences...
i will review if there is any concensus in that...
anyway now i when people should speak about it...
>
> Wouldn't it be clearer to build a list with all the sequences owned by
> the tables in istmt.objects, and then call ExecGrantStmt_oids() a single
> time with the big list?
>
at night i will see the code for this...
--
regards,
Jaime Casanova
Soporte y capacitación de PostgreSQL
Guayaquil - Ecuador
Cel. (593) 87171157
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-patches@postgresql.org
Cc: systemguards@gmail.com, alvherre@commandprompt.com, pgsql-hackers@postgresql.org
Subject: Re: [PATCHES] Extending grant insert on tables to sequences
In-Reply-To: <c2d9e70e0807080910m7cb65cd0i3ecb449ca8fed17d@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox