| From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Chapman Flack <chap(at)anastigmatix(dot)net>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Robert Haas <robertmhaas(at)gmail(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [HACKERS] AdvanceXLInsertBuffer vs. WAL segment compressibility |
| Date: | 2018-03-18 20:56:12 |
| Message-ID: | 46FFB930-B0F3-400A-85ED-D9948C97FADF@yesql.se |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 18 Mar 2018, at 03:09, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Chapman Flack <chap(at)anastigmatix(dot)net> writes:
>> Thanks for the review. I notice that cfbot has now flagged the patch as
>> failing, and when I look into it, it appears that cfbot is building with
>> your test patch, and without the xlog.c patch, and so the test naturally
>> fails. Does the cfbot require both patches to be attached to the same
>> email, in order to include them both?
>
> I believe so --- AFAIK it does not know anything about dependencies
> between different patches, and will just try to build whatever patch(es)
> appear in the latest email on a given thread. Munro might be able to
> provide more detail.
Right, I should’ve realized when I didn’t include your original patch as well,
sorry about that. Now we know that the proposed test fails without the patch
applied and clears with it, that was at least an interesting side effect =)
cheers ./daniel
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Isaac Morland | 2018-03-18 21:05:17 | Flexible permissions for REFRESH MATERIALIZED VIEW |
| Previous Message | Andres Freund | 2018-03-18 20:55:48 | ECPG installcheck tests fail if PGDATABASE is set |