Andres Freund wrote: > Ugh. I personally would say that's because that commit did stuff that we > normally trie hard not to do. While ColumnDef at least isn't serialized > into catalogs, we normally trie hard to break struct layout. Peter, > shouldn't that field at the very least have been added at the end?
FWIW we just noticed that because commit 699bf7d05 changed the heap_prepare_freeze_tuple() ABI back in December, it caused similar breakage :-( I didn't notice the implications when I read the patch. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services