Re: Add backup_type to pg_stat_progress_basebackup

On Tue, Jul 22, 2025 at 6:06 PM Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp> wrote:
>
> On Tue, 22 Jul 2025 17:48:35 +0900
> Shinya Kato <shinya11(dot)kato(at)gmail(dot)com> wrote:
>
> > Hi hackers,
> >
> > Starting with PostgreSQL 17, pg_basebackup supports incremental
> > backups. However, the pg_stat_progress_basebackup view doesn't
> > currently show the backup type (i.e., whether it's a full or
> > incremental backup).
> >
> > Therefore, I propose adding a backup_type column to this view. While
> > this information is available in pg_stat_activity, the backup type is
> > important for monitoring the progress of pg_basebackup, and including
> > it directly in the progress view would be very useful.
> >
> > Thoughts?
>
> That seems reasonable to me.
>
> Just one minor comment on the patch:
>
> +
> + <row>
> + <entry role="catalog_table_entry"><para role="column_definition">
> + <structfield>backup_type</structfield> <type>bigint</type>
> + </para>
> + <para>
> + Backup type. Either <literal>full</literal> or
> + <literal>incremental</literal>.
> + </para></entry>
> + </row>
>
> The type should be text rather than bigint.

Thank you for the review.
I made a careless mistake. Fixed.

--
Best regards,
Shinya Kato
NTT OSS Center