Re: Cleaning up nbtree after logical decoding on standby work - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Cleaning up nbtree after logical decoding on standby work
Date
Msg-id 20230608142155.ni7w6ca7le4w5zgb@alvherre.pgsql
Whole thread Raw
In response to Re: Cleaning up nbtree after logical decoding on standby work  (Peter Geoghegan <pg@bowt.ie>)
Responses Re: Cleaning up nbtree after logical decoding on standby work
List pgsql-hackers
On 2023-Jun-07, Peter Geoghegan wrote:

> On Wed, Jun 7, 2023 at 5:12 PM Andres Freund <andres@anarazel.de> wrote:

> > I don't really understand why the patch does s/heaprel/heapRel/.
> 
> That has been the style used within nbtree for many years now.

IMO this kind of change definitely does not have place in a post-beta1
restructuring patch.  We rarely indulge in case-fixing exercises at any
other time, and I don't see any good argument why post-beta1 is a better
time for it.  I suggest that you should strive to keep the patch as
small as possible.

-- 
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/
"Cómo ponemos nuestros dedos en la arcilla del otro. Eso es la amistad; jugar
al alfarero y ver qué formas se pueden sacar del otro" (C. Halloway en
La Feria de las Tinieblas, R. Bradbury)



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Let's make PostgreSQL multi-threaded
Next
From: Amit Langote
Date:
Subject: Re: generic plans and "initial" pruning