Robert Treat wrote:
certainly by the time 8.4 ships, these should work with freebsd I'd think.
ideally we would need to confirm this by release time, certainly getting a
bsd buildfarm member to compile with them would be a start (and very unlikely
to cause issues)
As soon as the DTrace port is working on FreeBSD, I will confirm that
the probes are working properly, and it's definitely a good idea to get
a buildfarm machine building with --enable-dtrace.
One thing I didnt understand after looking at this was...
* Probes to measure query time
query-parse-start (int, char *)
I would have guessed that the arguments might be pid and query string, but
looking at the probes, I see it defined as:
TRACE_POSTGRESQL_QUERY_PARSE_START(query_string);
which doesn't seem to match up... can you explain that piece?
Having the pid passed as an argument was my original intention, but it's
actually redundant since the pid is readily available from the script,
so I will fix the other probes with pid as args.
Overall, I like the probes you have breaking down query
parsing/planning/executing, though I like ours for measuring autovacuum
pieces, so I think the end game should be to just merge the two patches
together (barring any place there is direct conflict)... do you see any
issues with that?
Yes, to avoid confusion, the probes should be merged and resubmitted as
one patch. Have yours been ported to 8.4 yet? We also need to make sure
the names and arg types are consistent, probably should work on this
offline.
One other questions would be what to do with the dtrace scripts. I think
having a set of these available is a large boon for dtrace users, but do you
see that as something that needs to be distriubuted with the core?
I don't see the need to include the scripts with core now, maybe some
point in the future if it makes sense.
I'd lean
towards reviving the dtrace project on pgfoundry, but it might be worth
expanding the dynamic tracing chapter to include more examples and a pointer
to pgfoundry.
Agreed on both. I will add the new scripts to the dtrace project on
PgFoundry and add more info to the doc. I think you guys have some
interesting scripts as well that folks will find useful.
-Robert
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers