Re: [PATCH] Backport perl tests for pg_upgrade from 322becb60

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

In response to

Responses

Browse pgsql-hackers by date

  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