Re: dead code in pg_upgrade
От | Nathan Bossart |
---|---|
Тема | Re: dead code in pg_upgrade |
Дата | |
Msg-id | Z9hnua7Go5RgD0NT@nathan обсуждение исходный текст |
Ответ на | Re: dead code in pg_upgrade (Tom Lane <tgl@sss.pgh.pa.us>) |
Список | pgsql-hackers |
On Mon, Mar 17, 2025 at 01:56:49PM -0400, Tom Lane wrote: > Nathan Bossart <nathandbossart@gmail.com> writes: >> I noticed that the condition it's referring to >> if (strlen(old_cluster.tablespace_suffix) == 0) >> doesn't appear to have been reachable since support for upgrading from >> pre-9.2 was removed in v15 (commit e469f0a). Before then, this case seems >> to have only applied to upgrades from v8.4 or older versions. > >> I'm planning to commit the attached patch shortly after I double-check that >> I'm not missing anything. > > +1. e469f0aaf just removed code that was immediately connected to a > version check; I didn't look too hard for consequent simplifications. > After looking, I concur that tablespace_suffix can never be empty > anymore. I don't see any other spots checking this condition. Committed, thanks for looking. -- nathan
В списке pgsql-hackers по дате отправления: