Re: [PATCHES] Auto-explain patch

2008-03-29 Thread Simon Riggs
On Sat, 2008-03-29 at 08:47 +, Dean Rasheed wrote: > > I also think we should only log the EXPLAIN if we have logged the SQL > > statement. It's not much use on its own anyway. This then allows this > > feature to work neatly with log_statement and > > log_min_duration_statement. > > > > Woul

Re: [PATCHES] Auto-explain patch

2008-03-29 Thread Dean Rasheed
> On Mon, 2008-01-28 at 09:21 +, Dean Rasheed wrote: >> This is the patch allows logging of the explain plan for each query run, as >> described here: >> >> http://archives.postgresql.org/pgsql-performance/2008-01/msg00245.php >> >> I hope this is useful. > > This looks very good, though I do

Re: [PATCHES] Auto-explain patch

2008-03-27 Thread Simon Riggs
On Mon, 2008-01-28 at 09:21 +, Dean Rasheed wrote: > This is the patch allows logging of the explain plan for each query run, as > described here: > > http://archives.postgresql.org/pgsql-performance/2008-01/msg00245.php > > I hope this is useful. This looks very good, though I don't think

Re: [PATCHES] Auto-explain patch

2008-01-28 Thread David Fetter
On Mon, Jan 28, 2008 at 08:49:23PM +, Dean Rasheed wrote: > > > On Mon, Jan 28, 2008 at 07:55:53PM +, Dean Rasheed wrote: > >> > >>> Dean, > >>> > >>> Maybe I missed something obvious here, but how does this patch handle > >>> the situation where people have turned on INTEGER_DATETIMES? >

Re: [PATCHES] Auto-explain patch

2008-01-28 Thread Dean Rasheed
> Date: Mon, 28 Jan 2008 12:08:00 -0800 > From: [EMAIL PROTECTED] > To: [EMAIL PROTECTED] > CC: pgsql-patches@postgresql.org > Subject: Re: [PATCHES] Auto-explain patch > > On Mon, Jan 28, 2008 at 07:55:53PM +, Dean Rasheed wrote: >> >>> Dean, >>&

Re: [PATCHES] Auto-explain patch

2008-01-28 Thread David Fetter
On Mon, Jan 28, 2008 at 07:55:53PM +, Dean Rasheed wrote: > > > Dean, > > > > Maybe I missed something obvious here, but how does this patch handle > > the situation where people have turned on INTEGER_DATETIMES? > > > > Cheers, > > David. > > -- > > David Fetter http://fetter.org/ > > Phone:

Re: [PATCHES] Auto-explain patch

2008-01-28 Thread Dean Rasheed
> Dean, > > Maybe I missed something obvious here, but how does this patch handle > the situation where people have turned on INTEGER_DATETIMES? > > Cheers, > David. > -- > David Fetter http://fetter.org/ > Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter > Skype: davidfetter XMPP: [EMAIL P

Re: [PATCHES] Auto-explain patch

2008-01-28 Thread David Fetter
On Mon, Jan 28, 2008 at 09:21:50AM +, Dean Rasheed wrote: > > This is the patch allows logging of the explain plan for each query run, as > described here: > > http://archives.postgresql.org/pgsql-performance/2008-01/msg00245.php > > I hope this is useful. > > Dean. Dean, Maybe I missed