On Tue, Jan 16, 2024 at 9:37 PM Nathan Bossart <nathandboss...@gmail.com> wrote: > > The autoprewarm change (0003) does use this variable. I considered making > it optional (i.e., you could pass in NULL if you didn't want it), but I > didn't feel like the extra code in GetNamedDSMSegment() to allow this was > worth it so that callers could avoid creating a single bool.
I'm okay with it. The v8 patches look good to me. -- Bharath Rupireddy PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com