Re: abi-compliance-check failure due to recent changes to pg_{clear,restore}_{attribute,relation}_stats()
David E. Wheeler <david@justatheory.com>
From: "David E. Wheeler" <david@justatheory.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Bruce Momjian <bruce@momjian.us>,
Nathan Bossart <nathandbossart@gmail.com>,
pgsql-hackers@postgresql.org,
Andrew Dunstan <andrew@dunslane.net>,
Mankirat Singh <mankiratsingh1315@gmail.com>
Date: 2025-10-30T14:10:43Z
Lists: pgsql-hackers
On Oct 30, 2025, at 09:55, Tom Lane <tgl@sss.pgh.pa.us> wrote: > Trouble is, you then need an arbitrary client-made choice about which > commit to run the ABI check against. It’s currently coded to use the most recent tag or, if there is none in the branch, the branch root. > If that code does something we > realize we don't want, we're back up against the problem of moving the > buildfarm configuration to fix it. I'd rather the decision be opt-in. Fair. Just means that if no one adds a history file to a branch that branch will never be tested and there’s no automated way to realize it. > (Also, the only rules I heard proposed for such client-driven choices > involved git tags. I already explained why I don't want that: git > tags are hard to modify and subject to too many other constraints.) Yeah, it just went with the most recent tag to keep it simple, no other metadata. D