Re: Replication logging

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Simon Riggs <simon(at)2ndQuadrant(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Magnus Hagander <magnus(at)hagander(dot)net>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Replication logging
Date: 2011-01-19 16:59:14
Message-ID: 201101191659.p0JGxEd28921@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Simon Riggs wrote:
> On Tue, 2011-01-18 at 10:57 -0500, Tom Lane wrote:
> > Magnus Hagander <magnus(at)hagander(dot)net> writes:
> > > Is there *any* usecase for setting them differently though?
> >
> > I can't believe we're still engaged in painting this bikeshed. Let's
> > just control it off log_connections and have done.
>
> Yes, this is a waste of time. Leave it as is because its there for a
> very good reason, as Robert already explained.
>
> The code was put in explicitly because debugging replication connections
> is quite important and prior to that addition, wasn't easy. It's a very
> separate thing from logging the hundreds/thousands of other connections
> on the system.
>
> I don't really care about neatness of code, or neatness of parameters. I
> want to be able to confirm the details of the connection.
> pg_stat_replication is dynamic, not a historical record of connections
> and reconnections.
>
> How else will you diagnose an erratic network, or an accidental change
> of authority? Replication is so important it isn't worth tidying away a
> couple of lines of log. My objective is to make replication work, not to
> reduce the size of the average log file by 1 or 2 lines.
>
> I'm particularly concerned that people make such changes too quickly.
> There are many things in this area of code that need changing, but also
> many more that do not. If we are to move forwards we need to avoid going
> one step forwards, one step back.

There were enough people who wanted a change that we went ahead and did
it --- if there was lack of agreement, we would have delayed longer.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2011-01-19 17:01:09 Re: Re: patch: fix performance problems with repated decomprimation of varlena values in plpgsql
Previous Message Tom Lane 2011-01-19 16:52:27 Re: limiting hint bit I/O