From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | bruce(at)momjian(dot)us |
Cc: | Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM>, Robert Lor <Robert(dot)Lor(at)Sun(dot)COM>, pgsql-hackers list <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Additional DTrace Probes |
Date: | 2009-04-02 21:00:51 |
Message-ID: | 200904022100.n32L0pO00951@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
After talking to Tom, I have reverted this patch. It does not contain
documentation, and are not properly placed, and was submitted in
February. I was hoping it would be an easy addition but obviously not
so please resubmit for 8.5. Sorry.
---------------------------------------------------------------------------
bruce wrote:
>
> OK, updated patch attached and applied; thanks.
>
> ---------------------------------------------------------------------------
>
> Zdenek Kotala wrote:
> > It looks OK now.
> >
> > Zdenek
> >
> > Dne 8.03.09 21:58, Robert Lor napsal(a):
> >
> > >> 2) Maybe
> > >>
> > >> TRACE_POSTGRESQL_SLRU_READPAGE_PHYSICAL_DONE(true, -1, -1);
> > >>
> > >> would be better. Because slru_errcause, slru_errno can contains garbage
> > >> in situation when everything goes fine. Same for write.
> > >
> > > I've made the changes per your suggestion although one can argue that
> > > the script can check arg0, and if it's true, avoid using arg1 and arg2
> > > as they are meaningless.
> >
> > Thanks.
> >
> > >>
> > >> I think it is committable for 8.4.
> > >>
> > >
> > > That would be awesome!
> >
> > yeah, but it depends on commiters :-)
> >
> > Zdenek
> >
> > --
> > Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> > To make changes to your subscription:
> > http://www.postgresql.org/mailpref/pgsql-hackers
>
> --
> Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
> EnterpriseDB http://enterprisedb.com
>
> + If your life is a hard drive, Christ can be your backup. +
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2009-04-02 21:02:58 | Re: fix - function call with variadic parameter for type "any" |
Previous Message | Tom Lane | 2009-04-02 20:39:25 | Re: fix - function call with variadic parameter for type "any" |