Re: docs: note ownership requirement for refreshing materializedviews - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: docs: note ownership requirement for refreshing materializedviews
Date
Msg-id 20180816011534.GK3681@paquier.xyz
Whole thread Raw
In response to Re: docs: note ownership requirement for refreshing materializedviews  ("Jonathan S. Katz" <jonathan.katz@excoventures.com>)
Responses Re: docs: note ownership requirement for refreshing materializedviews
List pgsql-hackers
On Wed, Aug 15, 2018 at 09:06:34PM -0400, Jonathan S. Katz wrote:
> I played around with this feature a bit and did see this was the case.
> Also while playing around I noticed the error message was as such:
>
>     test=> REFRESH MATERIALIZED VIEW blah;
>     ERROR: must be owner of relation blah
>
> But it’s not a relation, it’s a materialized view. I attached a patch
> that I think should fix this. Kudos to Dave Cramer who was
> sitting next to me helping me to locate files and confirm assumptions.

A relation may be a materialized view, no?  The ACL check happens in
RangeVarCallbackOwnsTable by the way (look at ExecRefreshMatView in
matview.c).
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: "Jonathan S. Katz"
Date:
Subject: Re: docs: note ownership requirement for refreshing materializedviews
Next
From: "Shinoda, Noriyoshi (PN Japan GCS Delivery)"
Date:
Subject: Update comment in errcodes.txt correctly