Skip site navigation (1) Skip section navigation (2)

Re: Review: DTrace probes (merged version) ver_03

From: Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM>
To: Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM>
Cc: Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Robert Lor <Robert(dot)Lor(at)Sun(dot)COM>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Robert Treat <xzilla(at)users(dot)sourceforge(dot)net>, jesus(at)omniti(dot)com
Subject: Re: Review: DTrace probes (merged version) ver_03
Date: 2008-07-28 14:13:11
Message-ID: (view raw or whole thread)
Lists: pgsql-hackers
Zdenek Kotala napsal(a):
> Alvaro Herrera napsal(a):
>> Zdenek Kotala wrote:
>>> I performed review and I prepared own patch which contains only 
>>> probes without any issue. I suggest commit this patch because the 
>>> rest of patch is independent and it can be committed next commit fest 
>>> after rework.
>>> I found following issues:
>> I noticed that CLOG, Subtrans and Multixact probes are added during a
>> regular Checkpoint, but not during a shutdown flush.  I think the probes
>> should count that too (probably with the same counter).
> Yeah, good catch.

When I'm thinking about it, It seems to me better idea to have


Because you will able to determine what was a reason for flush and how log take 
shutting down.

By the way why the shutdown order is following:

05618     ShutdownCLOG();
05619     ShutdownSUBTRANS();
05620     ShutdownMultiXact();

What does happen when kill -9/power lost comes between lines 5617 and 5618?


In response to

pgsql-hackers by date

Next:From: Tom LaneDate: 2008-07-28 14:16:21
Subject: Re: Review: DTrace probes (merged version) ver_03
Previous:From: Andrew GierthDate: 2008-07-28 14:06:31
Subject: Re: WITH RECUSIVE patches 0723

Privacy Policy | About PostgreSQL
Copyright © 1996-2015 The PostgreSQL Global Development Group