Re: pgsql: Split out tiebreaker comparisons from comparetup_* functions

From: John Naylor <john(dot)naylor(at)enterprisedb(dot)com>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Cc: John Naylor <john(dot)naylor(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Split out tiebreaker comparisons from comparetup_* functions
Date: 2023-08-18 10:57:09
Message-ID: CAFBsxsHWouhoR4LAc1WDFvUWsnFKU4ehhpg5BJnqwOWBrjQ9wA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Fri, Aug 18, 2023 at 5:26 PM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
>
> > > Hammerkop is getting OOM errors with this change. Most grand totals
in the memory context stats show between 1 and 3 MB, and "tuplesort main"
only shows small-ish requests, so not sure what's going on.
> >
> > That remains the only new failure in the buildfarm, and it just failed
in the same way again. I'll try to figure out what could be the cause, but
if I don't I'll revert within 24 hours.
>
> I don't know what's going on on that machine but those two failures
> seem different, and there are also now some failures in release
> branches that don't have your commit, so I'd wait long for more
> evidence...

Ah, I did check at least one back branch yesterday and didn't see any
failures then. I see them now, thanks!

(FWIW, I've been looking for signs of memory malfeasance in the commit, and
found nothing yet)

--
John Naylor
EDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2023-08-18 14:58:51 Re: pgsql: Split out tiebreaker comparisons from comparetup_* functions
Previous Message Thomas Munro 2023-08-18 10:25:23 Re: pgsql: Split out tiebreaker comparisons from comparetup_* functions