Date: Tue, 18 Oct 2016 15:38:52 -0300
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Stephen Frost <sfrost@snowman.net>,
	Thomas Munro <thomas.munro@enterprisedb.com>,
	Andres Freund <andres@anarazel.de>, Evan Rempel <erempel@uvic.ca>,
	"pgsql-admin@postgresql.org" <pgsql-admin@postgresql.org>,
	Pg Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [HACKERS] 9.5 new setting "cluster name" and logging
Message-ID: <20161018183852.3grybpr3qbyyho33@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: 
 <CA+TgmoaAJMZ6kuapWbWPorz0wk5BqdTqWSkwD=_T1UZB3hYfEQ@mail.gmail.com>
User-Agent: NeoMutt/20161003 (1.7.0)
Precedence: bulk
Sender: pgsql-admin-owner@postgresql.org

Robert Haas wrote:
> On Mon, Oct 17, 2016 at 6:24 PM, Stephen Frost <sfrost@snowman.net> wrote:
> > I'm with Thomas on this and I disagree that the "csvlog bloat" argument
> > has merit.  If we're worried about bloat in csv then we should provide a
> > way for users to control what goes into the csvlog, not argue that
> > something which is clearly useful be excluded.
> 
> I agree, but I think if we do that it would be highly desirable do
> something to make the format discoverable.  For example, at the
> beginning of each file and whenever the format changes, we write some
> kind of unmistakable header line into the file identifying what fields
> will be present on each line thereafter.  It's undesirable for log
> analysis tools to need to contain logic that tries (imperfectly, no
> doubt) to reverse-engineer the field list.

Hmm, I wouldn't use the same file for different formats.  If the format
is changed, a different file should be used.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-admin mailing list (pgsql-admin@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-admin