pgsql: Remove the restriction that the relmap must be 512 bytes.

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Remove the restriction that the relmap must be 512 bytes.
Date: 2022-07-26 19:10:22
Message-ID: E1oGPwj-001NZa-73@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Remove the restriction that the relmap must be 512 bytes.

Instead of relying on the ability to atomically overwrite the
entire relmap file in one shot, write a new one and durably
rename it into place. Removing the struct padding and the
calculation showing why the map is exactly 512 bytes, and change
the maximum number of entries to a nearby round number.

Patch by me, reviewed by Andres Freund and Dilip Kumar.

Discussion: http://postgr.es/m/CA+TgmoZq5%3DLWDK7kHaUbmWXxcaTuw_QwafgG9dr-BaPym_U8WQ%40mail.gmail.com
Discussion: http://postgr.es/m/CAFiTN-ttOXLX75k_WzRo9ar=VvxFhrHi+rJxns997F+yvkm==A@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/d8cd0c6c95c0120168df93aae095df4e0682a08a

Modified Files
--------------
doc/src/sgml/monitoring.sgml | 4 +-
src/backend/utils/activity/wait_event.c | 4 +-
src/backend/utils/cache/relmapper.c | 94 +++++++++++++++++++--------------
src/include/utils/wait_event.h | 2 +-
4 files changed, 58 insertions(+), 46 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2022-07-26 19:14:36 pgsql: Fix brain fade in e530be2c5ce77475d56ccf8f4e0c4872b666ad5f.
Previous Message Robert Haas 2022-07-26 18:39:39 pgsql: Do not allow removal of superuser privileges from bootstrap user

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2022-07-26 19:19:45 Re: making relfilenodes 56 bits
Previous Message Robert Haas 2022-07-26 19:06:08 Re: let's disallow ALTER ROLE bootstrap_superuser NOSUPERUSER