On 11 December 2017 at 17:38, Tom Lane <t...@sss.pgh.pa.us> wrote: > Simon Riggs <si...@2ndquadrant.com> writes: >> On 11 December 2017 at 16:27, Tom Lane <t...@sss.pgh.pa.us> wrote: >>> For a *very* large majority of the callers of AllocSetContextCreate, >>> the context name is a simple C string constant, so we could just store >>> the pointer to it and save the space and cycles required to copy it. > >> Why have the string at all in that case? > > Try reading a MemoryContextStats dump without it ...
I understood. I thought you were suggesting removing it in favour of a pointer. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services