On 10.9.2014 20:25, Heikki Linnakangas wrote: > On 09/10/2014 01:49 AM, Tomas Vondra wrote: >> I also did a few 'minor' changes to the dense allocation patch, most >> notably: >> >> * renamed HashChunk/HashChunkData to MemoryChunk/MemoryChunkData >> The original naming seemed a bit awkward. > > That's too easy to confuse with regular MemoryContext and AllocChunk > stuff. I renamed it to HashMemoryChunk.
BTW this breaks the second patch, which is allocating new chunks when resizing the hash table. Should I rebase the patch, or can the commiter do s/MemoryChunk/HashMemoryChunk/ ? Assuming there are no more fixes needed, of course. Tomas -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
