Andres Freund <and...@anarazel.de> writes: > On 2017-12-19 13:17:52 -0500, Tom Lane wrote: >> Yeah. But please don't mess with MemoryContextStats per se --- >> I dunno about you guys but "call MemoryContextStats(TopMemoryContext)" >> is kinda wired into my gdb reflexes. I think what'd make sense >> is a new function "MemoryContextStatsTo(context, function_pointer)". >> It's okay to redefine the APIs of the per-context-type functions >> these would call, though, because nobody calls those functions directly.
> We already have MemoryContextStatsDetail() - it seems to make sense to > expand that API and leave MemoryContextStats() alone. I don't think > there's a need for a third variant? Sure, WFM. regards, tom lane