From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule? |
Date: | 2015-12-21 11:03:10 |
Message-ID: | 5677DC6E.7000703@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 17/12/15 19:07, Robert Haas wrote:
> On Mon, Dec 14, 2015 at 6:08 PM, Tomas Vondra
> <tomas(dot)vondra(at)2ndquadrant(dot)com> wrote:
>> So we know that we should expect about
>>
>> (prev_wal_bytes - wal_bytes) + (prev_wal_fpw_bytes - wal_fpw_bytes)
>>
>> ( regular WAL ) + ( FPW WAL )
>>
>> to be produced until the end of the current checkpoint. I don't have a clear
>> idea how to transform this into the 'progress' yet, but I'm pretty sure
>> tracking the two types of WAL is a key to a better solution. The x^1.5 is
>> probably a step in the right direction, but I don't feel particularly
>> confident about the 1.5 (which is rather arbitrary).
>
> If it works well empirically, does it really matter that it's
> arbitrary? I mean, the entire planner is full of fairly arbitrary
> assumptions about which things to consider in the cost model and which
> to ignore. The proof that we have made good decisions there is in the
> query plans it generates. (The proof that we have made bad decisions
> in some cases in the query plans, too.)
Agreed.
> I think a bigger problem for this patch is that Heikki seems to have
> almost completely disappeared.
Yeah, there's that problem too :-).
The reason I didn't commit this back then was lack of performance
testing. I'm fairly confident that this would be a significant
improvement for some workloads, and shouldn't hurt much even in the
worst case. But I did only a little testing on my laptop. I think Simon
was in favor of just committing it immediately, and Fabien wanted to see
more performance testing before committing.
I was hoping that Digoal would re-ran his original test case, and report
back on whether it helps. Fabien had a performance test setup, for
testing another patch, but he didn't want to run it to test this patch.
Amit did some testing, but didn't see a difference. We can take that as
a positive sign - no regression - or as a negative sign, but I think
that basically means that his test was just not sensitive to the FPW issue.
So Tomas, if you're willing to do some testing on this, that would be
brilliant!
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2015-12-21 11:34:57 | Re: Getting sorted data from foreign server for merge join |
Previous Message | Simon Riggs | 2015-12-21 10:58:36 | Re: Optimizing away second VACUUM heap scan when only BRIN indexes on table |