Horton

Re: table partitioning and access privileges

Fujii Masao <masao.fujii@oss.nttdata.com>

View thread View on postgr.es
From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Amit Langote <amitlangote09@gmail.com>
Cc: Fujii Masao <masao.fujii@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2020-01-30T15:54:40Z
Lists: pgsql-hackers 

On 2020/01/27 14:02, Amit Langote wrote:
> Fujii-san,
> 
> On Mon, Jan 27, 2020 at 11:19 AM Fujii Masao
> <masao.fujii@oss.nttdata.com> wrote:
>> On 2020/01/23 22:14, Fujii Masao wrote:
>>> Thanks for updating the patch! Barring any objection,
>>> I will commit this fix and backport it to all supported versions.
>>
>> Attached are the back-port versions of the patches.
>>
>>
>> The patch for master branch separates truncate_check_activity() into two
>> functions, but in v11 or before, truncate_check_activity() didn't exist and
>> its code was in truncate_check_rel(). So I had to write the back-port
>> version
>> of the patch for the previous versions and separate truncate_check_rel()
>> into three functions, i.e., truncate_check_rel(),
>> truncate_check_activity() and
>> truncate_check_perms().
> 
> Thank you for creating the back-port versions.  I agree with making
> the code look similar in all supported branches for the ease of future
> maintenance.

Thanks for the check! I pushed the patches.

Regards,

-- 
Fujii Masao
NTT DATA CORPORATION
Advanced Platform Technology Group
Research and Development Headquarters

Commits

  1. Make inherited LOCK TABLE perform access permission checks on parent table only.

  2. Add note about access permission checks by inherited TRUNCATE and LOCK TABLE.

  3. Revert commit 56bc82a511.

  4. Revert commit 606f350de9.

  5. Revert commit 928e755d22.

  6. Revert commit 4b96c03a0a.

  7. Revert commit a5b652f3a0.

  8. Revert commit de0177788b.

  9. Make inherited TRUNCATE perform access permission checks on parent table only.

  10. Regression tests for LOCK TABLE.