| From: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
|---|---|
| To: | 邱宇航 <iamqyh(at)gmail(dot)com> |
| Cc: | 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 08:00:03 |
| Message-ID: | CAN55FZ1S_e32YoKjeA9rco7+a9RpaB4vrFQJZfjXTu=JoiVJ3w@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On Thu, 27 Nov 2025 at 05:51, 邱宇航 <iamqyh(at)gmail(dot)com> wrote:
>
> > I do not think that will be a problem but I can change it if the
> > general consensus is towards this way. Also, if we change this for
> > pg_buffercache_mark_dirty_* functions, I think we need to apply the
> > same for the pg_buffercache_evict_* functions.
>
> After some testing, bgwriter/checkpointer didn' blocks the mark buffer
> dirty SQL. it's ok to use LWLockAcquire.
Thanks for testing this!
> There is an extra line break after elog(ERROR, "bad buffer ID: %d", buf)
> which can be removed.
Done in the v10. I wonder why pgindent did not catch this.
--
Regards,
Nazir Bilal Yavuz
Microsoft
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Mahendra Singh Thalor | 2025-11-27 08:15:12 | Re: Non-text mode for pg_dumpall |
| Previous Message | Nazir Bilal Yavuz | 2025-11-27 07:59:47 | Re: Add pg_buffercache_mark_dirty[_all] functions to the pg_buffercache |