Re: [COMMITTERS] pgsql: Add connection messages for streaming replication.

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Simon Riggs <sriggs(at)postgresql(dot)org>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Add connection messages for streaming replication.
Date: 2010-03-24 01:52:27
Message-ID: 3f0b79eb1003231852q3e9fabdfkfe29b2ff76e619bd@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Sat, Mar 20, 2010 at 4:19 AM, Simon Riggs <sriggs(at)postgresql(dot)org> wrote:
> Log Message:
> -----------
> Add connection messages for streaming replication. log_connections
> was broken for a replication connection and no messages were
> displayed on either standby or primary, at any debug level.
> Connection messages needed to diagnose session drop/reconnect
> events. Use LOG mode for now, discuss lowering in later releases.

LOG: connection authorized: user=foo database=replication

Currently, when the primary accepts the connection from the standby,
it emits the above message. But "database=replication" is not accurate
because no "database" is supplied by the standby unless it's explicitly
specified in primary_conninfo parameter. So, how about changing the
message as follow?

LOG: replication connection authorized: user=foo

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message User Itagaki 2010-03-24 02:05:16 pgstatsinfo - pg_statsinfo: Fix documentation about default
Previous Message David Fetter 2010-03-24 00:05:22 Re: pgsql: Typo fix from IRC breinbaas

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2010-03-24 02:17:31 Re: Proposal: access control jails (and introduction as aspiring GSoC student)
Previous Message Robert Haas 2010-03-24 01:51:32 Re: Proposal: access control jails (and introduction as aspiring GSoC student)