Re: Skip ExecCheckRTPerms in CTAS with no data
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Cc: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>, Anastasia Lubennikova <a.lubennikova@postgrespro.ru>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2020-11-17T01:32:16Z
Lists: pgsql-hackers
On Mon, Nov 16, 2020 at 04:01:33PM +0100, Peter Eisentraut wrote: > While this patch was nice enough to update the documentation about the > requirement of the INSERT privilege, this is maybe more confusing now: How > could a new table not have INSERT privilege? Yes, you can do that with > default privileges, but that's not well known and should be clarified in the > documentation. > > The SQL standard says that for CREATE TABLE AS, the INSERT "is effectively > executed without further Access Rule checking", which means the INSERT > privilege shouldn't be required at all. I suggest we consider doing that > instead. I don't see a use for the current behavior. Hmm. Is there anything specific for materialized views? They've checked for INSERT privileges at this phase since their introduction in 3bf3ab8c. -- Michael
Commits
-
Remove INSERT privilege check at table creation of CTAS and matview
- 878f3a19c6c8 14.0 landed
-
Relax INSERT privilege requirement for CTAS and matviews WITH NO DATA
- 846005e4f382 14.0 landed