Re: pgsql: Reduce scope of heap vacuum per_buffer_data - Mailing list pgsql-committers

From Melanie Plageman
Subject Re: pgsql: Reduce scope of heap vacuum per_buffer_data
Date
Msg-id CAAKRu_aGVuq3VUV4SoCOtAubMipY7u=X6sBXsUS+tRQ_+bAVuA@mail.gmail.com
Whole thread Raw
In response to pgsql: Reduce scope of heap vacuum per_buffer_data  (Melanie Plageman <melanieplageman@gmail.com>)
List pgsql-committers
On Tue, Feb 18, 2025 at 9:30 AM Melanie Plageman
<melanieplageman@gmail.com> wrote:
>
> Reduce scope of heap vacuum per_buffer_data
>
> Move lazy_scan_heap()'s per_buffer_data variable into a tighter scope.
> In lazy_scan_heap()'s phase I heap vacuuming, the read stream API
> returns a pointer to the next block number to vacuum. As long as
> read_stream_next_buffer() returns a valid buffer, per_buffer_data should
> always be valid.

Realized I forgot an important word in the commit message.
per_buffer_data doesn't point to the block number. It points to info
about the next block. Commit message should say "the read stream API
returns a pointer to information about the next block to vacuum". Oh
well... There is nothing I can do now.

- Melanie



pgsql-committers by date:

Previous
From: Melanie Plageman
Date:
Subject: pgsql: Reduce scope of heap vacuum per_buffer_data
Next
From: Jeff Davis
Date:
Subject: pgsql: Update outdated comments in nodeAgg.c.