Re: pg_upgrade analyze script

On Mon, Nov 9, 2020 at 11:22 AM Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>
> 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.

And now pushed, let's see if the buildfarm agrees with the Windows change...

--
Magnus Hagander
Me: https://www.hagander.net/
Work: https://www.redpill-linpro.com/