RE: Design of pg_stat_subscription_workers vs pgstats

From: "osumi(dot)takamichi(at)fujitsu(dot)com" <osumi(dot)takamichi(at)fujitsu(dot)com>
To: 'Masahiko Sawada' <sawada(dot)mshk(at)gmail(dot)com>
Cc: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: RE: Design of pg_stat_subscription_workers vs pgstats
Date: 2022-02-23 01:13:45
Message-ID: TYCPR01MB837314CD88ECC298874D5CCBED3C9@TYCPR01MB8373.jpnprd01.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tuesday, February 22, 2022 11:47 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> On Tue, Feb 22, 2022 at 9:22 PM osumi(dot)takamichi(at)fujitsu(dot)com
> <osumi(dot)takamichi(at)fujitsu(dot)com> wrote:
> > (4) doc/src/sgml/monitoring.sgml
> >
> > <row>
> > <entry role="catalog_table_entry"><para
> role="column_definition">
> > - <structfield>last_error_time</structfield> <type>timestamp with
> time zone</type>
> > + <structfield>sync_error_count</structfield> <type>uint8</type>
> > </para>
> > <para>
> > - Last time at which this error occurred
> > + Number of times the error occurred during the initial data
> > + copy
> > </para></entry>
> >
> > I supposed it might be better to use "initial data sync"
> > or "initial data synchronization", rather than "initial data copy".
>
> "Initial data synchronization" sounds like the whole table synchronization
> process including COPY and applying changes to catch up. But
> sync_error_count is incremented only during COPY so I used "initial data copy".
> What do you think?
Okay. Please keep it as is.

Best Regards,
Takamichi Osumi

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2022-02-23 01:48:55 Re: Race condition in InvalidateObsoleteReplicationSlots()
Previous Message Jacob Champion 2022-02-23 00:53:53 Re: pgcrypto: Remove internal padding implementation