Re: RelationGetNumberOfBlocks called before vacuum_get_cutoffs

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: Melanie Plageman <melanieplageman(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: RelationGetNumberOfBlocks called before vacuum_get_cutoffs
Date: 2025-06-01 18:22:38
Message-ID: CAH2-Wzkv6tKi0dzLTAT6g0mOLeHdvx+KtT9MeWam9SFwXpqdOA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Jun 1, 2025 at 1:51 PM Melanie Plageman
<melanieplageman(at)gmail(dot)com> wrote:
> Thanks for the report. That was a dumb mistake. There was no reason
> for me to move the line up as you can see in the diff -- it must have
> been unintentional.

I have to imagine that you moved rel_pages initialization back so that
it took place next to the initialization of other, similar BlockNumber
fields from LVRelState. IIRC I wrote a comment about this issue at
least in part because I understood the temptation to do that.

> I'll push the fix tomorrow.

Cool.

--
Peter Geoghegan

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Lakhin 2025-06-01 19:00:01 Re: Improving tracking/processing of buildfarm test failures
Previous Message Melanie Plageman 2025-06-01 17:51:06 Re: RelationGetNumberOfBlocks called before vacuum_get_cutoffs