public inbox for [email protected]  
help / color / mirror / Atom feed
From: Tomas Vondra <[email protected]>
To: [email protected]
Subject: pgsql: Revert "Improve accounting for memory used by shared hash tables
Date: Fri, 04 Apr 2025 02:50:11 +0000
Message-ID: <[email protected]> (raw)

Revert "Improve accounting for memory used by shared hash tables"

This reverts commit f5930f9a98ea65d659d41600a138e608988ad122.

This broke the expansion of private hash tables, which reallocates the
directory. But that's impossible when it's allocated together with the
other fields, and dir_realloc() failed with BogusFree. Clearly, this
needs rethinking.

Discussion: https://postgr.es/m/CAApHDvriCiNkm=v521AP6PKPfyWkJ++jqZ9eqX4cXnhxLv8w-A@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1aff1dc8dfa7584ddbc695e1aae0f767db66326f

Modified Files
--------------
src/backend/storage/ipc/shmem.c   |   4 +-
src/backend/utils/hash/dynahash.c | 281 +++++++++-----------------------------
src/include/utils/hsearch.h       |   3 +-
3 files changed, 66 insertions(+), 222 deletions(-)



reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: pgsql: Revert "Improve accounting for memory used by shared hash tables
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox