| From: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Xuneng Zhou <xunengzhou(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Aidar Imamov <a(dot)imamov(at)postgrespro(dot)ru>, Joseph Koshakow <koshy44(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Add pg_buffercache_mark_dirty[_all] functions to the pg_buffercache |
| Date: | 2025-11-27 07:59:47 |
| Message-ID: | CAN55FZ0DWe5X-mVMoQVez4z4+iM7=JUPW9XxayrN-fkg4SgtAQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On Thu, 27 Nov 2025 at 02:35, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Mon, Nov 24, 2025 at 10:48:24AM +0300, Nazir Bilal Yavuz wrote:
> > Thanks for the heads up! It is rebased, I also added
> > 'CHECK_FOR_INTERRUPTS()' while looping over the 'NBuffers' regarding
> > the comment in the email [1].
> >
> > [1] https://postgr.es/m/D5BB1D85-0F2A-419F-A7B1-426505525D3A%40gmail.com
>
> One thing that would make more sense to me is to
> split the patch in two: one for the changes in bufmgr.c/h and one for
> pg_buffercache. There can be an argument that these new APIs could be
> useful for out-of-core code, as well, to let developers play with the
> state of the buffers. I mean, why not, the thread is also about API
> extensibility, as well, to enforce dirty states. :)
>
> Any thoughts or comments from others?
I agree with you, the patches make more sense this way.
The patches are split into two in v10. There are no changes from v9,
except that one extra blank line was removed [1].
[1] https://postgr.es/m/188562F6-5BBB-49AB-B9E1-6312AE7970E8%40gmail.com
--
Regards,
Nazir Bilal Yavuz
Microsoft
| Attachment | Content-Type | Size |
|---|---|---|
| v10-0001-Add-internal-APIs-for-marking-buffers-dirty-effi.patch | text/x-patch | 7.6 KB |
| v10-0002-Add-pg_buffercache_mark_dirty-_relation-_all-fun.patch | text/x-patch | 18.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nazir Bilal Yavuz | 2025-11-27 08:00:03 | Re: Add pg_buffercache_mark_dirty[_all] functions to the pg_buffercache |
| Previous Message | Amul Sul | 2025-11-27 07:57:46 | Re: Refactoring: Use soft error reporting for *_opt_overflow functions of date/timestamp |