From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | "Anton A(dot) Melnikov" <aamelnikov(at)inbox(dot)ru> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: [PATCH] Backport perl tests for pg_upgrade from 322becb60 |
Date: | 2022-12-09 05:19:00 |
Message-ID: | Y5LFRAoX8FvqnmKW@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Aug 01, 2022 at 01:02:21AM +0300, Anton A. Melnikov wrote:
> As far as i understand from this thread: https://www.postgresql.org/message-id/flat/Yox1ME99GhAemMq1%40paquier.xyz,
> the aim of the perl version for the pg_upgrade tests is to achieve equality of dumps for most cross-versions cases.
> If so this is the significant improvement as previously in test.sh resulted dumps retained unequal and the user
> was asked to eyeball them manually during cross upgrades between different major versions.
> So, the backport of the perl tests also seems preferable to me.
I don't really agree with that. These TAP tests are really new
development, and it took a few tries to get them completely right
(well, as much right as it holds for HEAD). If we were to backport
any of this, there is a risk of introducing a bug in what we do with
any of that, potentially hiding a issue critical related to
pg_upgrade. That's not worth taking a risk for.
Saying that, I agree that more needs to be done, but I would limit
that only to HEAD and let it mature more into the tree in an
incremental fashion.
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Hayato Kuroda (Fujitsu) | 2022-12-09 05:19:37 | RE: Time delayed LR (WAS Re: logical replication restrictions) |
Previous Message | Michael Paquier | 2022-12-09 04:36:03 | Re: add \dpS to psql |