Re: ExecRTCheckPerms() and many prunable partitions

Alvaro Herrera <alvherre@alvh.no-ip.org>

From: Alvaro Herrera <alvherre@alvh.no-ip.org>

To: Amit Langote <amitlangote09@gmail.com>

Cc: Ian Lawrence Barwick <barwick@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, David Rowley <dgrowleyml@gmail.com>, Greg Stark <stark@mit.edu>, Julien Rouhaud <rjuju123@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>

Date: 2022-12-07T11:44:15Z

Lists: pgsql-hackers

On 2022-Dec-07, Amit Langote wrote:

> While doing that, I noticed that I had missed updating at least one
> comment which still says that permission checking is done off of the
> range table.  Attached patch fixes that.

Pushed, thanks.

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/

Commits

Add a test case for a316a3bc
- 054ff3b33a85 17.0 landed
- 0a14bca662a4 16.0 landed
Correctly set userid of subquery relations' child rels
- a316a3bc6d3f 16.0 landed
Fix buggy recursion in flatten_rtes_walker().
- c7468c73f7b6 16.0 landed
Remove some dead code in selfuncs.c
- 438e6b724090 16.0 landed
Update outdated comment in ApplyRetrieveRule
- 29861e228a47 16.0 landed
Rework query relation permission checking
- a61b1f74823c 16.0 landed
Generalize ri_RootToPartitionMap to use for non-partition children
- fb958b5da86d 16.0 landed
Stop accessing checkAsUser via RTE in some cases
- 599b33b9492d 16.0 landed
Add 'missing_ok' argument to build_attrmap_by_name
- ad86d159b6ab 16.0 landed