Failed transaction statistics to measure the logical replication progress - Mailing list pgsql-hackers

From osumi.takamichi@fujitsu.com
Subject Failed transaction statistics to measure the logical replication progress
Date
Msg-id OSBPR01MB48887CA8F40C8D984A6DC00CED199@OSBPR01MB4888.jpnprd01.prod.outlook.com
Whole thread Raw
Responses Re: Failed transaction statistics to measure the logical replication progress  (vignesh C <vignesh21@gmail.com>)
Re: Failed transaction statistics to measure the logical replication progress  (Ajin Cherian <itsajin@gmail.com>)
Re: Failed transaction statistics to measure the logical replication progress  (Masahiko Sawada <sawada.mshk@gmail.com>)
List pgsql-hackers
Hello, hackers


When the current HEAD fails during logical decoding, the failure
increments txns count in pg_stat_replication_slots - [1] and adds
the transaction size to the sum of bytes in the same repeatedly
on the publisher, until the problem is solved.
One of the good examples is duplication error on the subscriber side
and this applies to both streaming and spill cases as well.

This update prevents users from grasping the exact number and size of
successful and unsuccessful transactions. Accordingly, we need to
have new columns of failed transactions that will work to differentiate
both of them for all types, which means spill, streaming and normal
transactions. This will help users to measure the exact status of
logical replication.

Attached file is the POC patch for this.
Current design is to save failed stats data in the ReplicationSlot struct.
This is because after the error, I'm not able to access the ReorderBuffer object.
Thus, I chose the object where I can interact with at the ReplicationSlotRelease timing.

Below is one example that I can get on the publisher,
after the duplication error on the subscriber caused by insert is solved.

postgres=# select * from pg_stat_replication_slots;
-[ RECORD 1 ]-------+------
slot_name           | mysub
spill_txns          | 0
spill_count         | 0
spill_bytes         | 0
failed_spill_txns   | 0
failed_spill_bytes  | 0
stream_txns         | 0
stream_count        | 0
stream_bytes        | 0
failed_stream_txns  | 0
failed_stream_bytes | 0
total_txns          | 4
total_bytes         | 528
failed_total_txns   | 3
failed_total_bytes  | 396
stats_reset         |


Any ideas and comments are welcome.

[1] - https://www.postgresql.org/docs/devel/monitoring-stats.html#MONITORING-PG-STAT-REPLICATION-SLOTS-VIEW


Best Regards,
    Takamichi Osumi


Attachment

pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: bugfix: when the blocksize is 32k, the function page_header of pageinspect returns negative numbers.
Next
From: Michael Paquier
Date:
Subject: Re: [PATCH] Pull general SASL framework out of SCRAM