Re: Add pg_stat_recovery system view - Mailing list pgsql-hackers

From Chao Li
Subject Re: Add pg_stat_recovery system view
Date
Msg-id 500B406F-4968-4F33-9E21-02C0E50904B8@gmail.com
Whole thread Raw
In response to Re: Add pg_stat_recovery system view  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers

> On Mar 9, 2026, at 14:06, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Mon, Mar 09, 2026 at 01:20:45PM +0800, Chao Li wrote:
>>> [1] https://www.postgresql.org/docs/devel/error-message-reporting.html
>>
>> Would it make sense to mention this distinction in the header
>> comments for ereport and elog in the code? For these logging APIs, I
>> think developers are often more likely to read the code comments
>> than the documentation.
>
> As of the link already mentioned:
> "Therefore, elog should be used only for internal errors .."
> --
> Michael

Yes, the doc has clearly explained everything. I meant to ask if we want to add a bit explanation to the header comment
ofelog and ereport. Currently elog’s comment only says: 
```
/*----------
* Old-style error reporting API: to be used in this way:
*        elog(ERROR, "portal \"%s\" not found", stmt->portalname);
*----------
```
That sounds elog is just old, but somewhat equivalent to ereport.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/







pgsql-hackers by date:

Previous
From: Chao Li
Date:
Subject: Re: ALTER TABLE: warn when actions do not recurse to partitions
Next
From: Xuneng Zhou
Date:
Subject: Re: Refactor recovery conflict signaling a little