From: | Dean Rasheed <dean_rasheed(at)hotmail(dot)com> |
---|---|
To: | Alex Hunsaker <badalex(at)gmail(dot)com> |
Cc: | <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Auto-explain patch |
Date: | 2008-07-03 16:58:07 |
Message-ID: | BAY102-W42AF4FE40B20BE3C18EE73F2980@phx.gbl |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Here is an updated version of the patch, with a debug_explain_min_duration
parameter to allow explaining of just slow-running queries. I've also incorporated
a couple of Simon Riggs' suggestions for formatting the output better.
Do I need to post this to -patches, or is that now obsolete?
Regards, Dean
----------------------------------------
> From: dean_rasheed(at)hotmail(dot)com
> To: badalex(at)gmail(dot)com
> CC: pgsql-hackers(at)postgresql(dot)org
> Subject: RE: [HACKERS] Auto-explain patch
> Date: Wed, 2 Jul 2008 19:42:06 +0000
>
>
>> Its certainly not useful to *me* in its current form. It would
>> produce way to much (usless) output. However if it were tied to
>> log_min_duration_statement so I get auto explains for long running
>> queries... That would be very useful indeed. Even if it has to
>> explain everything just to toss out the explain if it did not meet
>> log_min_duration_statement. Unless I missed something and thats
>> exactly what it does?
>
> Thanks for the feedback. I agree, it does need a way to limit the
> output, and target just the slow-running queries.
>
> I also remember now the problem I had last time:- since this debug
> output is produced at a lower level than the other statement logging
> (so it can explain *all* SQL executed, not just top-level statements), it
> is difficult to control using the normal statement logging parameters.
>
> It would be easy to add another parameter, debug_explain_min_duration,
> specific to this option, to limit it to slow low-level queries.
>
> This would allow setting debug_explain_min_duration to be smaller than
> log_min_duration_statement, which makes sense, since the latter
> controls logging of top-level statements which may result in multiple
> low-level queries.
>
> Doing it this way would mean instrumenting all queries, but only
> explaining the slow ones, when debug_explain_plan is on.
> I'll have a play and see how it goes...
>
> Regards, Dean
>
> _________________________________________________________________
> Live Search Charades - guess correctly and find hidden videos
> http://www.searchcharades.com/
_________________________________________________________________
The next generation of Windows Live is here
http://www.windowslive.co.uk/get-live
Attachment | Content-Type | Size |
---|---|---|
auto-explain2.patch | text/x-patch | 13.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Mark Mielke | 2008-07-03 17:18:19 | Re: A Windows x64 port of PostgreSQL |
Previous Message | Alvaro Herrera | 2008-07-03 16:53:15 | Re: PATCH: CITEXT 2.0 |