| From: | Paul Kim <mok03127(at)gmail(dot)com> |
|---|---|
| To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Cc: | aekorotkov(at)gmail(dot)com, michael(at)paquier(dot)xyz, x4mmm(at)yandex-team(dot)ru, Paul Kim <mok03127(at)gmail(dot)com> |
| Subject: | Re: Restore vacuum_delay_point() in GIN posting-tree leaf vacuum |
| Date: | 2026-07-23 02:26:35 |
| Message-ID: | 20260723022635.21756-1-mok03127@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2026-07-22, Alexander Korotkov wrote:
> Please, check the patch attached.
Thanks, Alexander. The patch looks good to me -- it's the same change
I posted upthread, and the placement is right: the call sits after the
page has been unlocked and released, so no buffer content lock (nor any
other LWLock) is held across the delay, matching the sibling loops in
ginbulkdelete() and ginvacuumcleanup(). The added comment reads well.
One note for the back-branches, since it's marked Backpatch-through:
14: vacuum_delay_point() only gained its bool argument in e5b0b0ce150
(v18), so on 14-17 the call would need to be the no-argument
vacuum_delay_point().
--
Paul Kim
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Min, Baohong | 2026-07-23 02:39:10 | [PATCH] Reduce LWLockWaitListLock() cache-line contention with adaptive spin reads |
| Previous Message | Quan Zongliang | 2026-07-23 02:23:24 | Re: [PATCH] Remove unused include from analyze.c |