On 2023-Nov-09, Amit Kapila wrote:
> These comments appear mostly repetitive to what is already mentioned
> in start_postmaster(). So, I have changed those referred to already
> written comments, and slightly adjusted the comments at another place.
> See attached.
I'd still rather mention check_old_cluster_for_valid_slots() just above
the Assert() in InvalidatePossiblyObsoleteSlot(). It looks too bare to
me otherwise.
> Personally, I don't see the need for a test for this, so removed the
> same but can add it back if you or others think so.
I'm neutral on having a test for this. I'm not sure this is easy to
break unintentionally. OTOH the test is cheap, since it only has to run
pg_upgrade itself and not, say, another initdb. On the (as Robert says)
third hand, would we have tests for each possible GUC that we'd like not
to be changed during pg_upgrade? I suspect not, which suggests we don't
want this one either.
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"El Maquinismo fue proscrito so pena de cosquilleo hasta la muerte"
(Ijon Tichy en Viajes, Stanislaw Lem)