Re: Fix for REFRESH MATERIALIZED VIEW ownership error message

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Dave Cramer <pg(at)fastcrypt(dot)com>
Cc: "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix for REFRESH MATERIALIZED VIEW ownership error message
Date: 2018-08-17 23:35:12
Message-ID: 20180817233512.3ksiy7pnbn7mjfz7@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dave Cramer 2018-08-17 23:39:04 Re: Fix for REFRESH MATERIALIZED VIEW ownership error message
Previous Message Dave Cramer 2018-08-17 23:32:09 Re: Fix for REFRESH MATERIALIZED VIEW ownership error message