Re: Add bump memory context type and use it for tuplesorts - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Add bump memory context type and use it for tuplesorts
Date
Msg-id 20240407230456.jux6li4cg23tnqbt@awork3.anarazel.de
Whole thread Raw
In response to Re: Add bump memory context type and use it for tuplesorts  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Add bump memory context type and use it for tuplesorts
List pgsql-hackers
Hi,

On 2024-04-08 00:55:42 +0200, Daniel Gustafsson wrote:
> > On 8 Apr 2024, at 00:41, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > 
> > Tomas Vondra <tomas.vondra@enterprisedb.com> writes:
> >> Yup, changing it to this:
> > 
> >> #define KeeperBlock(set) ((BumpBlock *) ((char *) (set) +
> >> MAXALIGN(sizeof(BumpContext))))
> > 
> >> fixes the issue for me.
> > 
> > Mamba is happy with that change, too.
> 
> Unrelated to that one, seems like turaco ran into another issue:
> 
> running bootstrap script ... TRAP: failed Assert("total_allocated == context->mem_allocated"), File: "bump.c", Line:
808,PID: 7809
 
> 
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=turaco&dt=2024-04-07%2022%3A42%3A54

What makes you think that's unrelated? To me that looks like the same issue?

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Cutting support for OpenSSL 1.0.1 and 1.0.2 in 17~?
Next
From: David Rowley
Date:
Subject: Re: Add bump memory context type and use it for tuplesorts