On Tue, Dec 19, 2017 at 8:44 AM, Craig Ringer <cr...@2ndquadrant.com> wrote: > I didn't want to mess with the MemoryContextMethods and expose a > printf-wrapper style typedef in memnodes.h, so I went with a hook global.
That looks pretty grotty to me. I think if you want to elog/ereport this, you need to pass another argument to MemoryContextStats() or add another memory context method. This is pretty much a textbook example of the wrong way to use a global variable, IMHO. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
