From: | Jeff Janes <jeff(dot)janes(at)gmail(dot)com> |
---|---|
To: | Greg Smith <greg(at)2ndquadrant(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Instrument checkpoint sync calls |
Date: | 2010-11-30 20:15:55 |
Message-ID: | AANLkTinxHedsbMCq4fQT1Lx-iBed8QewmskpujWmJ4yA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Nov 30, 2010 at 8:38 AM, Greg Smith <greg(at)2ndquadrant(dot)com> wrote:
Hi Greg,
Thanks for the update.
> This might be ready for some proper review now. I know there's at least one
> blatant bug still in here I haven't found yet, related to how the averages
> are computed.
Yes, the blatant bug:
average_sync_time = CheckpointStats.ckpt_longest_sync /
CheckpointStats.ckpt_sync_rels;
That should clearly be ckpt_agg_sync_time, not ckpt_longest_sync.
> I saw this once:
>
> LOG: checkpoint complete: wrote 0 buffers (0.0%); 0 transaction log file(s)
> added, 0 removed, 1 recycled; write=0.000 s, sync=0.000 s, total=0.001 s;
> sync files=0, longest=0.000 s, average=-9223372036854775808.-2147483 s
>
> After an immediate checkpoint, so at least one path not quite right yet.
Not clear what the right thing to do here is. I guess we should
special case the div by zero to yield zero for the average?
The patch is in unified diff rather than context diff. Doesn't bother
me, but the wiki on doing a review says...
Cheers,
Jeff
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2010-11-30 20:16:57 | Re: DELETE with LIMIT (or my first hack) |
Previous Message | Josh Berkus | 2010-11-30 20:09:31 | Where are we on Standby Promotion? |