On Tue, Oct 26, 2010 at 11:55:07AM +0900, Itagaki Takahiro wrote:
> On Tue, Oct 26, 2010 at 11:34 AM, David Fetter <david(at)fetter(dot)org> wrote:
> >> Do we need to 'add' it?
> > Possibly. My understanding is that it couldn't really replace it.
> Ah, I see. I was wrong. We can have modification privileges for
> views even if they have no INSTEAD OF triggers.
> So, I think your original patch is the best solution. We could use
> has_table_privilege() additionally, but we need to consider any
> other places if we use it. For example, DROP privileges, etc.
That seems like a matter for a separate patch. Looking this over, I
found I'd created a query that can never get used, so please find
enclosed the next version of the patch :)
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
In response to
pgsql-hackers by date
|Next:||From: Alvaro Herrera||Date: 2010-10-29 15:40:47|
|Subject: Re: crash in plancache with subtransactions|
|Previous:||From: Tom Lane||Date: 2010-10-29 15:23:38|
|Subject: Re: plan time of MASSIVE partitioning ... |