On Fri, 17 Aug 2018 at 19:35, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote:
> On 2018-Aug-17, Dave Cramer wrote: > > > The only place this is used is in aclcheck_error > > case OBJECT_MATVIEW: > > msg = gettext_noop("permission denied for materialized view %s"); > > break; > > Yes, but do we pass RefreshMatViewStmt->relkind to that routine? I > don't see that we do. Maybe I misread the code. > Actually the code path that gets executed is: case OBJECT_MATVIEW: msg = gettext_noop("must be owner of materialized view %s"); break; as I have the patch applied and now see: \c - test You are now connected to database "test" as user "test". test=> refresh materialized view b; ERROR: must be owner of materialized view b Dave Cramer