Re: pgsql: Add pg_stat_archiver statistics view.

From: Andres Freund <andres(at)2ndquadrant(dot)com>
To: Fujii Masao <fujii(at)postgresql(dot)org>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Add pg_stat_archiver statistics view.
Date: 2014-02-03 21:20:52
Message-ID: 20140203212052.GB6729@awork2.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Hi,

On 2014-01-28 17:59:12 +0000, Fujii Masao wrote:
> Add pg_stat_archiver statistics view.
>
> This view shows the statistics about the WAL archiver process's activity.
>
> Gabriele Bartolini, reviewed by Michael Paquier, refactored a bit by me.

As coverity notest the following lines of code:
if (archiver_stats->last_failed_wal == 0)
nulls[4] = true;
else
values[4] = CStringGetTextDatum(archiver_stats->last_failed_wal);
won't work as intended since last_failed_wal is an array of chars and
thus will never be 0 (which should be NULL if so).

Greetings,

Andres Freund

--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2014-02-03 21:38:42 Re: pgsql: Document a few more regression test hazards.
Previous Message Tom Lane 2014-02-03 19:47:33 pgsql: Fix *-qualification of named parameters in SQL-language function