On Wed, Nov 12, 2025 at 9:54 AM vignesh C <[email protected]> wrote: > > Hi, > > This patch proposes renaming a column in the view > pg_stat_subscription_stats to disambiguate between table sync and > sequence sync error counts. With the introduction of > seq_sync_error_count, the existing column name sync_error_count > becomes ambiguous, it's unclear whether it refers to tables, > sequences, or both. Renaming it to tbl_sync_error_count makes the > meaning explicit: it represents errors that occurred during table > synchronization. This helps users avoid confusion when interpreting > the view's output.
+1 on the intent. It will definitely help avoid confusion in the names. I’m slightly leaning toward using 'table_sync_error_count' instead of 'tbl_sync_error_count'. The name 'table_sync_error_count' isn’t too long and should work fine, while 'tbl_..' feels more like an internal variable name. But I would like to see what others prefer here. thanks Shveta
