> On Nov 13, 2025, at 17:52, vignesh C <[email protected]> wrote:
>
> On Thu, 13 Nov 2025 at 09:12, Amit Kapila <[email protected]> wrote:
>>
>> On Thu, Nov 13, 2025 at 4:31 AM Peter Smith <[email protected]> wrote:
>>>
>>> CURRENT
>>> seq_sync_error_count
>>> sync_error_count
>>>
>>> MY SUGGESTION
>>> sync_seq_error_count
>>> sync_table_error_count
>>>
>>
>> +1. Along with consistency with other column names, it will also make
>> it easier for users to group and query all 'sync' related counts
>> together.
>
> This seems better, the attached v2 version patch has the changes for the same.
>
> Regards,
> Vignesh
> <v2-0001-Rename-sync_error_count-to-sync_table_error_count.patch>
V2 overall looks good, a small comment is that the commit message need to be
updated:
```
Subject: [PATCH v2] Rename sync_error_count to sync_table_error_count in
subscription statistics
This patch renames the sync_error_count column to sync_table_error_count in
the pg_stat_subscription_stats view. The new name makes the purpose explicit
```
We now actually rename two fields.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
