On Sat, Jun 1, 2019 at 3:09 PM Michael Paquier <mich...@paquier.xyz> wrote: > I am fine to live with the dependency with vacuum.h as it is not that > strange. However for BulkInsertState we get a hard dependency with a > heap-related area and it seems to me that we had better move that part > out of heapam.c, as we want a clear dependency cut with the heap AM > for any new custom table AM.
Yeah, I noticed this, too. +1 for doing something about it. Not sure exactly what is the best approach. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
