Hi,
On 2014-02-21 14:15:09 +0100, Christian Kruse wrote:
> +/* ----------
> + * pgstat_fetch_stat_local_beentry() -
> + *
> + * Like pgstat_fetch_stat_beentry() but with local addtions (like xid and
> + * xmin values of the backend)
s/local addtions/locally computed addititions/
> +/* ----------
> + * LocalPgBackendStatus
> + *
> + * When we build the backend status array, we use LocalPgBackendStatus to be
> + * able to add new values to the struct when needed without adding new fields
> + * to the shared memory. It contains the backend status as a first member.
> + * ----------
> + */
> +typedef struct LocalPgBackendStatus
> +{
> + /*
> + * Local version of the backend status entry
> + */
> + PgBackendStatus backendStatus;
> +
> + /*
> + * The xid of the current transaction if available, InvalidTransactionId
> + * if not
> + */
> + TransactionId backend_xid;
> +
> + /*
> + * The xmin of the current session if available, InvalidTransactionId
> + * if not
> + */
> + TransactionId backend_xmin;
> +} LocalPgBackendStatus;
> +
Those are sentences, so they should include a . at the end.
I think this is ready for committer.
Greetings,
Andres Freund
-- Andres Freund http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training &
Services