On Fri, Dec 12, 2008 at 02:51, ITAGAKI Takahiro
> Here is an updated version of pg_stat_statements.
I took the liberty of splitting this out into 3 patches for easier review:
-adds 2 new hooks (startup_hook and shutdown_hook).
-fixes missing process_shared_preload_libraries on EXEC_BACKEND
-changes int pgstat_track_activity_query_size to PGDLLIMPORT int
-addes sourceText to QueyDesc and modifies callers to populate it
-adds new guc explain_analyze_format to print out statistic counters
-adds rtime/utime to explain analyze printout
-moves statistic counters (ReadBufferCount etc...) into struct
Instrumentation via a new struct BufferCounter (allows modules like
the proposed easy access)
the contrib module
A few comments:
Is there a reason you add sourceText to QueryDesc? AFAICT you can do
ActivePortal->sourceText and it will always be populated correctly.
I think the explain_analyze_format guc is a clever way of getting
around the explain analyze verbose you proposed earlier. But I dont
see any doc updates for it.
Im still not overly fond of the "statistics." custom guc name, but
what can you do...
Other than that it looks good, though I admit I have not had the time
to sit down and thoroughly test it yet...
In response to
pgsql-hackers by date
|Next:||From: Zdenek Kotala||Date: 2008-12-22 07:43:32|
|Subject: Re: reloptions and toast tables|
|Previous:||From: Hitoshi Harada||Date: 2008-12-22 04:57:42|
|Subject: Re: Is "Window" really safe as a global typedef name?|