From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Georgios Kokolatos <gkokolatos(at)protonmail(dot)com>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pg_upgrade analyze script |
Date: | 2020-11-09 10:22:56 |
Message-ID: | CABUevEwi8-zPTrYFUZod36ZOt73kmR19yiajN4dB8cFVWp=5rg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Nov 9, 2020 at 8:53 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Mon, Nov 02, 2020 at 02:18:32PM +0100, Magnus Hagander wrote:
> > On Fri, Oct 30, 2020 at 5:10 PM Georgios Kokolatos
> > <gkokolatos(at)protonmail(dot)com> wrote:
> >> I did notice that the cfbot [1] is failing for this patch.
> >> Please try to address the issues for the upcoming Commitfest.
> >
> > Thanks for the notice -- PFA a rebased version!
>
> No objections to remove this script from me.
>
> I have spotted one small-ish thing. This patch is missing to update
> the following code in vcregress.pl:
> print "\nSetting up stats on new cluster\n\n";
> system(".\\analyze_new_cluster.bat") == 0 or exit 1;
Ah, nice catch -- thanks! I guess this is unfortunately not a test
that's part of what cfbot tests.
Untested on Windows, but following the patterns of the rows before it.
I will go ahead and push this version in a bit.
--
Magnus Hagander
Me: https://www.hagander.net/
Work: https://www.redpill-linpro.com/
Attachment | Content-Type | Size |
---|---|---|
pg_upgrade_analyze.patch | text/x-patch | 8.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bharath Rupireddy | 2020-11-09 10:41:15 | Re: Multi Inserts in CREATE TABLE AS - revived patch |
Previous Message | Peter Eisentraut | 2020-11-09 10:10:52 | Re: default result formats setting |