Re: Adding argument names to aggregate functions

From: Jim Jones <jim(dot)jones(at)uni-muenster(dot)de>
To: Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Adding argument names to aggregate functions
Date: 2023-04-18 10:39:33
Message-ID: cd7c78aa-ec48-3e40-a7f9-edf77ff5742e@uni-muenster.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 18.04.23 12:27, Dagfinn Ilmari Mannsåker wrote:
> Link to the actual job:
> https://cirrus-ci.com/task/5881376021413888
>
> The failure was:
>
> [09:54:38.727] 216/262 postgresql:recovery / recovery/031_recovery_conflict ERROR 198.73s exit status 60
>
> Looking at its log:
>
> https://api.cirrus-ci.com/v1/artifact/task/5881376021413888/testrun/build/testrun/recovery/031_recovery_conflict/log/regress_log_031_recovery_conflict
>
> we see:
>
> timed out waiting for match: (?^:User was holding a relation lock for too long) at /Users/admin/pgsql/src/test/recovery/t/031_recovery_conflict.pl line 311.
>
> That looks indeed completely unrelated to this patch.

Yes, that's what I suspected. The patch passes all tests now :)

I've marked the CF entry as "Ready for Committer".

Jim

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2023-04-18 12:32:37 Re: [PATCH] Add `verify-system` sslmode to use system CA pool for server cert
Previous Message Dagfinn Ilmari Mannsåker 2023-04-18 10:27:54 Re: Adding argument names to aggregate functions