Re: SLRUs in the main buffer pool - Page Header definitions

From: Andres Freund <andres(at)anarazel(dot)de>
To: "Bagga, Rishu" <bagrishu(at)amazon(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, "Debnath, Shawn" <sdn(at)amazon(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>
Subject: Re: SLRUs in the main buffer pool - Page Header definitions
Date: 2023-02-06 20:30:38
Message-ID: 20230206203038.czsdqdzrqfgfqcwl@awork3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

> From 098f37c0a17fc32a94bff43817414e01fcfb234f Mon Sep 17 00:00:00 2001
> From: Rishu Bagga <bagrishu(at)amazon(dot)com>
> Date: Thu, 15 Sep 2022 00:55:25 +0000
> Subject: [PATCH] slru to buffercache + page headers + upgrade
>
> ---
> contrib/amcheck/verify_nbtree.c | 2 +-
> [...]
> 65 files changed, 2176 insertions(+), 3258 deletions(-)

Unfortunately a quite large patch, with this little explanation, is hard to
review. I could read through the entire thread to try to figure out what this
is doing, but I really shouldn't have to.

You're changing quite fundamental APIs across the tree. Why is that required
for the topic at hand? Why is it worth doing that, given it'll break loads of
extensions?

Greetings,

Andres Freund

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2023-02-06 20:34:49 Re: Exit walsender before confirming remote flush in logical replication
Previous Message Nikita Malakhov 2023-02-06 20:21:45 Re: Pluggable toaster