Re: Add pg_stat_recovery system view
Chao Li <li.evan.chao@gmail.com>
From: Chao Li <li.evan.chao@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Tom Lane <tgl@sss.pgh.pa.us>,
Xuneng Zhou <xunengzhou@gmail.com>,
yangyz <1197620467@qq.com>,
pgsql-hackers <pgsql-hackers@lists.postgresql.org>
Date: 2026-03-09T06:57:03Z
Lists: 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 of elog 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/
Commits
-
Fix order of columns in pg_stat_recovery
- 2d4ead6f4bd0 19 (unreleased) landed
-
Add system view pg_stat_recovery
- 01d485b142e4 19 (unreleased) landed
-
Refactor code retrieving string for RecoveryPauseState
- 42a12856a6a0 19 (unreleased) landed
-
Move definition of XLogRecoveryCtlData to xlogrecovery.h
- 5f8124a0cf33 19 (unreleased) landed