On Thu, 2021-05-06 at 17:24 -0700, Jeff Davis wrote: > It's the table AM's responsibility to detoast out-of-line datums and > toast any values that are too large (see > heapam.c:heap_prepare_insert()).
Do we have general agreement on this point? Did I miss another purpose of detoasting in tablecmds.c, or can we just remove that part of the patch? Regards, Jeff Davis