Hi again, Just a quick follow-up.
> (*) BTW I noticed a mistake in the commented code. The condition > should be `>=`, not `<`, i.e: > > ``` > if(new_cluster.controldata.cat_ver >= SLRU_SEG_FILENAMES_CHANGE_CAT_VER) > return; > ``` The concentration of caffeine in my blood is a bit low right now. I suspect I may need to re-check this statement with a fresh head. Also it occured to me that as a 4th option we could just get rid of this check. Users however will pay the price every time they execute pg_upgrade so I doubt we are going to do this. -- Best regards, Aleksander Alekseev
