From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pg_upgrade dead code for pre-8.4 versions |
Date: | 2020-10-06 18:45:21 |
Message-ID: | 20201006184521.GA12277@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Oct 6, 2020 at 11:40:57AM +0200, Daniel Gustafsson wrote:
> > On 6 Oct 2020, at 11:27, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>
> > ISTM that code could be cleaned out, because it should be dead based on the initial check that we are upgrading from 8.4 or later?
>
> +1. Commit 2209b3923a7afe0b removed support for 8.3 so anything left checking
> for pre-8.4 should be removed as a pre-8.4 source cluster couldn't be upgraded
> with pg_upgrade today.
OK, fixed with the attached patch, applied to all branches. There was
code that tested for >= 8.4 (useless test) or < 8.4 (dead code). I
also adjusted the version tests to be consistent.
--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EnterpriseDB https://enterprisedb.com
The usefulness of a cup is in its emptiness, Bruce Lee
Attachment | Content-Type | Size |
---|---|---|
pg_upgrade.diff | text/x-diff | 3.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2020-10-06 19:05:09 | Re: pg_upgrade analyze script |
Previous Message | Andres Freund | 2020-10-06 18:34:50 | Re: Add primary keys to system catalogs |