From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Magnus Hagander <magnus(at)hagander(dot)net> |
Subject: | Re: pgsql: Add 'basebackup_to_shell' contrib module. |
Date: | 2022-03-30 18:07:21 |
Message-ID: | CA+hUKG+qAr45GGOdwpS6FjooOiBtbYX3GPPfb8=UbccnTZnv6A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Thu, Mar 31, 2022 at 5:23 AM Andres Freund <andres(at)anarazel(dot)de> wrote:
> On 2022-03-26 13:51:35 -0700, Andres Freund wrote:
> > Prototype patch attached.
>
> Because I forgot about cfbot when attaching the patch, cfbot actually ran with
> it under this thread's cf entry. It does look like an improvement to me:
> https://cirrus-ci.com/task/6397292629458944?logs=test_world#L900
>
> We certainly can do better, but it's sufficiently better than what we have
> right now. So I'd like to commit it?
Nice, this will save a lot of time scrolling around trying to figure
out what broke.
> > Would we want to do this in all branches? I'd vote for yes, but ...
>
> Unless somebody speaks in favor of doing this across branches, I'd just go for
> HEAD.
I don't see any reason not to do it on all branches. If anyone is
machine-processing the output and cares about format changes they will
be happy about the improvement.
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2022-03-30 18:27:57 | pgsql: Add range_agg with multirange inputs |
Previous Message | Robert Haas | 2022-03-30 17:16:47 | Re: pgsql: Add 'basebackup_to_shell' contrib module. |
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Davis | 2022-03-30 18:31:40 | Re: Logical insert/update/delete WAL records for custom table AMs |
Previous Message | Greg Stark | 2022-03-30 17:35:57 | Re: Add proper planner support for ORDER BY / DISTINCT aggregates |