On Dec 8, 2009, at 5:10 AM, Zdenek Kotala wrote:
> Dne 8.12.09 00:27, Bernd Helmle napsal(a):
>> --On 13. November 2009 23:29:41 +0100 Zdenek Kotala <Zdenek.Kotala@Sun.COM> wrote:
>>> t contains two DTrace probe groups. One is related to monitoring SLRU
>>> and second is about executor nodes.
>>>
>>> I merged it with the head.
>>>
>>> Original end of mail thread is here:
>>>
>>> http://archives.postgresql.org/pgsql-hackers/2009-04/msg00148.php
>> I've started to review this.
>> It seems to me the attached patch wasn't adjusted or discussed again to address Tom's complaints? At least the
executorprobes contained here hold still the same issues mentioned by Tom in the discussion linked here.
>
> I did not make any change. I only revival patch and merge it with head. I think that SLRU probes are OK and
acceptable.
>
> Tom's issues with executor probes are still there and I expect discussion about them. IIRC Theo uses these probes in
production.
>
> If you think that it is better I could split patch into two separate patches and both can be reviewed separately.
I suppose I see it as a simple thing. The probes have no performance impact when they are not instrumented. I've used
themon rare occasion to understand which exec nodes are causing which disk accesses. Seemed pretty darn useful at the
time. There is (of course) some performance overhead when they are enabled, but that is intentionally performed by the
operator,so it seems like a non-issue.
Now, there was some indication that there was a better place to probe that would be more comprehensive -- that should
beaddressed.
--
Theo Schlossnagle
Esoteric Curio -- http://lethargy.org/
OmniTI Computer Consulting, Inc. -- http://omniti.com/
