From: | Rachel Heaton <rachelmheaton(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Rewriting the test of pg_upgrade as a TAP test - take three - remastered set |
Date: | 2021-09-07 21:43:15 |
Message-ID: | CADJcwiVeBQu7X-cvoSCWpYvFwC_kwMphX-=HgJ4Sn9X3n6dTZQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello Michael,
This patch needs the update from 201a76183 -- the function `get_new_node`
no longer exists.
Running check tests in the pg_upgrade folder fails for this reason.
Thank you,
Rachel
On Tue, Sep 7, 2021 at 2:06 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Tue, May 18, 2021 at 10:49:39AM +0900, Michael Paquier wrote:
> > Makes sense. For now, I'll update this patch set so as it is possible
> > to use custom dumps, as an option in parallel of pg_regress when
> > specifying a different source code path. I'll also decouple the
> > business with probin updates and stick with the approach used by the
> > buildfarm code.
>
> This has proved to not be that difficult. With the updated version
> attached, pg_upgrade has two modes to set up the old instance used for
> the upgrade with older binaries:
> - With oldsrc and oldinstall set, pg_regress gets used, same way as
> HEAD.
> - With olddump and oldinstall set, an old dump is loaded instead in
> the old instance before launching the upgrade.
>
> oldsrc and olddump are exclusive options. Similarly to HEAD, the
> dumps taken from the old and new instances generate diffs that can be
> inspected manually. The updates of probin are done without any
> dependencies to the source path of the old instance, copying from the
> buildfarm.
>
> While on it, I have fixed a couple of things that exist in test.sh but
> were not reflected in this new script:
> - Handling of postfix operations with ~13 clusters.
> - Handling oldstyle_length for ~9.6 clusters.
> - Handling of EXTRA_REGRESS_OPT.
>
> This stuff still needs to be expanded depending on how PostgresNode is
> made backward-compatible, but I'll wait for that to happen before
> going further down here. I have also spent some time testing all that
> with MSVC, and the installation paths used for pg_regress&co make the
> script a tad more confusing, so I have dropped this part for now.
>
> Thanks,
> --
> Michael
>
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2021-09-07 22:00:48 | Re: Bug in query rewriter - hasModifyingCTE not getting set |
Previous Message | Victor Spirin | 2021-09-07 21:40:11 | Re: Atomic rename feature for Windows. |