Re: Add os_page_num to pg_buffercache
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Cc: Mircea Cadariu <cadariu.mircea@gmail.com>, pgsql-hackers@lists.postgresql.org
Date: 2025-11-24T05:35:32Z
Lists: pgsql-hackers
On Sun, Nov 23, 2025 at 09:15:31AM +0000, Bertrand Drouvot wrote: > Attached a rebase due to 7d9043aee80. Also 0003 has a minor change (as compared > to v8-0004) to avoid this error when creating the 1.6 version with the new code: Yes, sorry, I forgot to mention that part. I have played with the patch for a couple of hours, fixed a couple of issues, rewording and tweaking things while browsing the whole (typedefs.list was incorrect, docs were partially incorrect), and applied the result. I did not see a point in 0001, as well, because the refactored "internal" function we'd have just one caller for the proposed macro and function. The original function pg_buffercache_numa_pages could be dropped when upgrading to v1.7 now that the view pg_buffercache_numa relies on the new SQL function pg_buffercache_os_pages(boolean), but I could not be really excited about that.. We could add a DROP FUNCTION, of course. By the way, thanks for the effort of splitting up things. This was super useful for the review and when dealing with each part of the proposed changes. -- Michael
Commits
-
pg_buffercache: Add pg_buffercache_os_pages
- 4b203d499c61 19 (unreleased) landed
-
doc: Fix style of description for pg_buffercache_numa.os_page_num
- fce13424b9c5 19 (unreleased) landed
-
Add CHECK_FOR_INTERRUPTS in contrib/pg_buffercache functions.
- eab9e4e27c0c 19 (unreleased) cited
-
pg_walsummary: Improve stability of test checking statistics
- a27893df45ec 19 (unreleased) cited