Re: Fix and improve allocation formulas

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Álvaro Herrera <alvherre(at)kurilemu(dot)de>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Fix and improve allocation formulas
Date: 2025-12-24 20:19:32
Message-ID: aUxK1KCQ5l1dYgq_@momjian.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Dec 11, 2025 at 05:56:13PM +0100, Álvaro Herrera wrote:
> On 2025-Dec-11, Andres Freund wrote:
>
> > a) changing over everything at once is worth the backpatch hazard and review
> > pain
>
> The other issue with these giant patches is that they cause many largish
> patches waiting in the commitfest process to require rebases, which are
> sometimes not trivial to do. Also, all the Postgres forks will
> require tedious merges later on.
>
> I have my part of blame for having committed the mass change to
> XLogRecPtrIsValid in a2b02293bc65. I'm starting to regret that now.

I think the bigger issue is that these patches bypass the normal
workflow of implementing changes in Postgres --- specifically, asking
about Desirability first:

https://wiki.postgresql.org/wiki/Todo
Desirability -> Design -> Implement -> Test -> Review -> Commit

It would have been much cleaner to discuss the desirability of this
change on its own.

--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com

Do not let urgent matters crowd out time for investment in the future.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Matthias van de Meent 2025-12-24 20:52:18 Re: Add --extra-dependencies and immediate data dumping for pg_dump/pg_upgrade
Previous Message Sami Imseih 2025-12-24 20:16:38 Re: Cleaning up PREPARE query strings?