On Tue, 24 Dec 2024 at 11:19, David Rowley <dgrowle...@gmail.com> wrote: > The attached adjusts that Assert code so that a fresh CompactAttribute > is populated instead of modifying the TupleDesc's one. I'm not sure > if populate_compact_attribute_internal() is exactly the nicest way to > do this. I'll think a bit harder about that. Assume the attached is > POC grade.
I've now pushed a fix for this using the same method but with the code factored around a little differently. I didn't want to expose the populate_compact_attribute_internal() function externally, so I invented verify_compact_attribute() to call from TupleDescCompactAttr(). Thanks for the report. David
