Re: Add last_commit_lsn to pg_stat_database

2024-06-04 Thread Kyotaro Horiguchi
At Tue, 20 Feb 2024 07:56:28 +0900, Michael Paquier wrote in > On Mon, Feb 19, 2024 at 10:26:43AM +0100, Tomas Vondra wrote: > It just means that I am not much a fan of the signature changes done > for RecordTransactionCommit() and AtEOXact_PgStat_Database(), adding a > dependency to a specify L

Re: Add last_commit_lsn to pg_stat_database

2024-05-28 Thread James Coleman
On Mon, Feb 19, 2024 at 3:56 PM Michael Paquier wrote: > > On Mon, Feb 19, 2024 at 10:26:43AM +0100, Tomas Vondra wrote: > > On 2/19/24 07:57, Michael Paquier wrote: > > > On Sun, Feb 18, 2024 at 02:28:06AM +0100, Tomas Vondra wrote: > >>> 1) Do we really need to modify RecordTransactionCommitPrep

Re: Add last_commit_lsn to pg_stat_database

2024-05-28 Thread James Coleman
On Thu, Mar 7, 2024 at 10:30 AM Heikki Linnakangas wrote: > > > I've previously noted in "Add last commit LSN to > > pg_last_committed_xact()" [1] that it's not possible to monitor how > > many bytes of WAL behind a logical replication slot is (computing such > > is obviously trivial for physical

Re: Add last_commit_lsn to pg_stat_database

2024-04-08 Thread Kirill Reshke
Hi James, There are some review in the thread that need to be addressed. it seems that we need to mark this entry "Waiting on Author" and move to the next CF [0]. Thanks [0] https://commitfest.postgresql.org/47/4355/ On Sat, 10 Jun 2023 at 05:27, James Coleman wrote: > I've previously noted i

Re: Add last_commit_lsn to pg_stat_database

2024-04-05 Thread Cary Huang
The following review has been posted through the commitfest application: make installcheck-world: tested, passed Implements feature: tested, passed Spec compliant: not tested Documentation:not tested Hello I think it is convenient to know the last commit LSN of a data

Re: Add last_commit_lsn to pg_stat_database

2024-03-07 Thread Heikki Linnakangas
I've previously noted in "Add last commit LSN to pg_last_committed_xact()" [1] that it's not possible to monitor how many bytes of WAL behind a logical replication slot is (computing such is obviously trivial for physical slots) because the slot doesn't need to replicate beyond the last commit. In

Re: Add last_commit_lsn to pg_stat_database

2024-02-19 Thread Michael Paquier
On Mon, Feb 19, 2024 at 10:26:43AM +0100, Tomas Vondra wrote: > On 2/19/24 07:57, Michael Paquier wrote: > > On Sun, Feb 18, 2024 at 02:28:06AM +0100, Tomas Vondra wrote: >>> 1) Do we really need to modify RecordTransactionCommitPrepared and >>> XactLogCommitRecord to return the LSN of the commit r

Re: Add last_commit_lsn to pg_stat_database

2024-02-19 Thread Tomas Vondra
On 2/19/24 07:57, Michael Paquier wrote: > On Sun, Feb 18, 2024 at 02:28:06AM +0100, Tomas Vondra wrote: >> Thanks for the updated patch. I don't have a clear opinion on the >> feature and whether this is the way to implement it, but I have two >> simple questions. > > Some users I know of would b

Re: Add last_commit_lsn to pg_stat_database

2024-02-18 Thread Michael Paquier
On Sun, Feb 18, 2024 at 02:28:06AM +0100, Tomas Vondra wrote: > Thanks for the updated patch. I don't have a clear opinion on the > feature and whether this is the way to implement it, but I have two > simple questions. Some users I know of would be really happy to be able to get this information

Re: Add last_commit_lsn to pg_stat_database

2024-02-17 Thread Tomas Vondra
Hi James, Thanks for the updated patch. I don't have a clear opinion on the feature and whether this is the way to implement it, but I have two simple questions. 1) Do we really need to modify RecordTransactionCommitPrepared and XactLogCommitRecord to return the LSN of the commit record? Can't we

Re: Add last_commit_lsn to pg_stat_database

2024-01-22 Thread James Coleman
On Sun, Jan 21, 2024 at 10:26 PM Peter Smith wrote: > > 2024-01 Commitfest. > > Hi, This patch has a CF status of "Needs Review", but it seems like > there was some CFbot test failure last time it was run [1]. Please > have a look and post an updated version if necessary. > > == > [1] > https

Re: Add last_commit_lsn to pg_stat_database

2024-01-21 Thread Peter Smith
2024-01 Commitfest. Hi, This patch has a CF status of "Needs Review", but it seems like there was some CFbot test failure last time it was run [1]. Please have a look and post an updated version if necessary. == [1] https://cirrus-ci.com/github/postgresql-cfbot/postgresql/commitfest/46/4355

Re: Add last_commit_lsn to pg_stat_database

2024-01-17 Thread James Coleman
On Sun, Jan 14, 2024 at 6:01 AM vignesh C wrote: > > On Sat, 10 Jun 2023 at 07:57, James Coleman wrote: > > > > I've previously noted in "Add last commit LSN to > > pg_last_committed_xact()" [1] that it's not possible to monitor how > > many bytes of WAL behind a logical replication slot is (comp

Re: Add last_commit_lsn to pg_stat_database

2024-01-17 Thread James Coleman
Hello, Thanks for reviewing! On Tue, Sep 19, 2023 at 8:16 AM Aleksander Alekseev wrote: > > Hi, > > > [...] > > As I was thinking about how to improve things, I realized that this > > information (since it's for monitoring anyway) fits more naturally > > into the stats system. I'd originally tho

Re: Add last_commit_lsn to pg_stat_database

2024-01-14 Thread vignesh C
On Sat, 10 Jun 2023 at 07:57, James Coleman wrote: > > I've previously noted in "Add last commit LSN to > pg_last_committed_xact()" [1] that it's not possible to monitor how > many bytes of WAL behind a logical replication slot is (computing such > is obviously trivial for physical slots) because

Re: Add last_commit_lsn to pg_stat_database

2023-09-19 Thread Aleksander Alekseev
Hi, > [...] > As I was thinking about how to improve things, I realized that this > information (since it's for monitoring anyway) fits more naturally > into the stats system. I'd originally thought of exposing it in > pg_stat_wal, but that's per-cluster rather than per-database (indeed, > this is

Add last_commit_lsn to pg_stat_database

2023-06-09 Thread James Coleman
I've previously noted in "Add last commit LSN to pg_last_committed_xact()" [1] that it's not possible to monitor how many bytes of WAL behind a logical replication slot is (computing such is obviously trivial for physical slots) because the slot doesn't need to replicate beyond the last commit. In