On Tue, Feb 09, 2021 at 04:27:34PM +0900, Michael Paquier wrote:
> Putting sanity checks within all the table_* functions of tableam.h
> would not be a good idea, as nothing prevents the call of what's
> stored in rel->rd_tableam.
I have been playing with this idea, and finished with the attached,
which is not the sexiest patch around. The table AM used as fallback
for tables without storage is called no_storage (this could be called
virtual_am?). Reverting e786be5 or dd705a0 leads to an error coming
from no_storage instead of a crash.
One thing to note is that this simplifies a bit slot_callbacks as
views, foreign tables and partitioned tables can grab their slot type
directly from this new table AM.
--
Michael