Re: Inconsistency in vacuum behavior - Mailing list pgsql-hackers

From Alexander Pyhalov
Subject Re: Inconsistency in vacuum behavior
Date
Msg-id 2cca954df24b106b462bc7c2259cbd83@postgrespro.ru
Whole thread Raw
In response to Re: Inconsistency in vacuum behavior  (Nikita Malakhov <hukutoc@gmail.com>)
Responses Re: Inconsistency in vacuum behavior
List pgsql-hackers
Nikita Malakhov писал 2023-01-16 17:26:
> Hi!
> 
> Here's the patch that fixes this case, please check it out.
> The patch adds vacuum_is_permitted_for_relation() check before adding
> partition relation to the vacuum list, and if permission is denied the
> relation
> is not added, so it is not passed to vacuum_rel() and there are no try
> to
> acquire the lock.
> 
> Cheers!

Hi.

The patch seems to solve the issue.
Two minor questions I have:
1) should we error out if HeapTupleIsValid(part_tuple) is false?
2) comment "Check partition relations for vacuum permit" seems to be 
broken in some way.

-- 
Best regards,
Alexander Pyhalov,
Postgres Professional



pgsql-hackers by date:

Previous
From: Ashutosh Bapat
Date:
Subject: Re: Logical replication timeout problem
Next
From: Tom Lane
Date:
Subject: Re: Make EXPLAIN generate a generic plan for a parameterized query