Re: Assertion failure with small block sizes

Gregory Stark <stark(at)enterprisedb(dot)com> writes:
> "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
>> Hmm. I'm inclined to reverse the tests (there are 3 not just 1) in
>> heapam.c, so that it explicitly tries to toast only in plain tables,
>> rather than adding more exclusion cases. Thoughts?

> Well RELKIND_UNCATALOGED can be usefully toasted in that data can be
> compressed internally.

But by the time we are inserting any data that needs compression, the
relkind should not be that anymore. This would only be an issue if
pg_proc.h itself contained DATA() lines long enough to need toasting.
I argue that that isn't true and isn't likely to become true. (See
ts_debug() for an example of deliberately avoiding such a case...)

regards, tom lane