On Thu, Jan 10, 2019 at 7:05 PM Andres Freund <and...@anarazel.de> wrote: > ISTM that the first block would best belong into new files like > access/rel[ation].h and access/common/rel[ation].h.
+1. > I think the second > set should be renamed to be table_open() (with backward compat > redirects, there's way way too many references) and should go into > access/table.h access/table/table.c alongside tableam.[ch], Sounds reasonable. > but I could > also see just putting them into relation.[ch]. I would view that as a less-preferred alternative, but not crazy. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company