Re: Restore support for USE_ASSERT_CHECKING in extensions only
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <dgrowleyml@gmail.com>
Cc: Andrew Kane <andrew@ankane.org>, pgsql-hackers@postgresql.org
Date: 2025-01-10T23:48:08Z
Lists: pgsql-hackers
David Rowley <dgrowleyml@gmail.com> writes: > On Sat, 11 Jan 2025 at 12:32, Tom Lane <tgl@sss.pgh.pa.us> wrote: >> No, this completely fails to address the problem. The concern is >> that the extension has been compiled under USE_ASSERT_CHECKING, >> so it will try to call the function. If the function's not there >> in core, kaboom. > hmm, you got me confused. Maybe you missed that the extension will be > the one compiling the static inline TupleDescCompactAttr() function > and will use the macro instead? No, I don't think I missed anything. Inline function or no, the extension will contain a call to verify_compact_attribute(), and that won't work if the core backend doesn't have that function. Depending on what platform you test on, the extension might have to actually reach that function call before it fails. regards, tom lane
Commits
-
Make verify_compact_attribute available in non-assert builds
- 34c6e652425f 18.0 landed