Re: Eagerly evict bulkwrite strategy ring

From: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
To: Melanie Plageman <melanieplageman(at)gmail(dot)com>
Cc: Kirill Reshke <reshkekirill(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Andres Freund <andres(at)anarazel(dot)de>
Subject: Re: Eagerly evict bulkwrite strategy ring
Date: 2025-09-12 07:16:14
Message-ID: CAN55FZ0CTPnDdNszwDKpfdACHDhY12ArybH4smyD_QEJ+msQZg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Fri, 12 Sept 2025 at 02:17, Melanie Plageman
<melanieplageman(at)gmail(dot)com> wrote:
>
> While looking at this, I decided it didn't make sense to have sweep
> variables in the strategy object, so I've actually changed the way
> StrategySweepNextBuffer() works. There was also an issue with the
> sweep -- it could run into and past the starting buffer. So, I had to
> change it. Take a look at the new method and let me know what you
> think.

It looks good to me. StrategySweepNextBuffer()'s comment is no longer
correct, though.

--
Regards,
Nazir Bilal Yavuz
Microsoft

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-09-12 07:23:05 Add support for entry counting in pgstats
Previous Message Zsolt Parragi 2025-09-12 07:02:16 Re: OAuth client code doesn't work with Google OAuth