From: Tom Lane <tgl@sss.pgh.pa.us>
To: Chapman Flack <chap@anastigmatix.net>
cc: Stephen Frost <sfrost@snowman.net>,
        Tomas Vondra <tomas.vondra@2ndquadrant.com>,
        Michael Paquier <michael@paquier.xyz>,
        Daniel Gustafsson <daniel@yesql.se>,
        Michael Paquier <michael.paquier@gmail.com>,
        Heikki Linnakangas <hlinnaka@iki.fi>,
        Robert Haas <robertmhaas@gmail.com>, Bruce Momjian <bruce@momjian.us>,
        PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [HACKERS] AdvanceXLInsertBuffer vs. WAL segment compressibility
In-reply-to: <5ABEC2C0.5040207@anastigmatix.net>
References: <46FFB930-B0F3-400A-85ED-D9948C97FADF@yesql.se>
 <5AAEE00E.9050503@anastigmatix.net>
 <21C25ED8-4ED8-4AD4-B551-3E3987ECB18F@yesql.se>
 <5AAEFF11.3070903@anastigmatix.net>
 <20180326032731.GM24540@tamriel.snowman.net>
 <20180326054354.GB2759@paquier.xyz> <5ABAC668.5080203@anastigmatix.net>
 <8c00b26a-d6da-5764-e67e-9c4d8128081a@2ndquadrant.com>
 <5ABD9083.5050908@anastigmatix.net> <12915.1522433926@sss.pgh.pa.us>
 <20180330182423.GS24540@tamriel.snowman.net> <24353.1522441284@sss.pgh.pa.us>
 <5ABEC2C0.5040207@anastigmatix.net>
Comments: In-reply-to Chapman Flack <chap@anastigmatix.net>
	message dated "Fri, 30 Mar 2018 19:05:36 -0400"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <11425.1522451462.1@sss.pgh.pa.us>
Date: Fri, 30 Mar 2018 19:11:02 -0400
Message-ID: <11426.1522451462@sss.pgh.pa.us>
Precedence: bulk

Chapman Flack <chap@anastigmatix.net> writes:
> On 03/30/18 16:21, Tom Lane wrote:
>> I did not like the proposed test case too much, particularly not its
>> undocumented API change for check_pg_config,

> Other than that API change, was there something the test case could have
> done differently to make you like it more?

Well, if that'd been properly documented I'd probably have pushed it
without complaint.  But I did wonder whether it could've been folded
into one of the existing tests of pg_switch_wal().  This doesn't seem
like a property worth spending a lot of cycles on testing.

			regards, tom lane