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