From: | Greg Stark <gsstark(at)mit(dot)edu> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Gregory Stark <gsstark(at)mit(dot)edu>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Martijn van Oosterhout <kleptog(at)svana(dot)org>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Ending EXPLAIN ANALYZE early (was Re: That EXPLAIN ANALYZE patch still needs work) |
Date: | 2006-06-08 19:39:24 |
Message-ID: | 87ejxzjv2r.fsf@stark.xeocode.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> Gregory Stark <gsstark(at)mit(dot)edu> writes:
> > Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> > Would it be possible to make a whole new protocol message for EXPLAIN results?
>
> I'm really unwilling to get into that. For one thing, that would
> absolutely positively break *all* use of EXPLAIN from un-fixed clients,
> whether you were trying to stop early or not.
Well I was picturing still having the manual EXPLAIN ANALYZE which works as it
does now replacing the query results in addition to a lower level protocol
request which generates a protocol response with the data.
> > The reason I'm suggesting that is because it might make it easier to implement
> > the SIGINFO handler that sends incremental EXPLAIN results on demand that I
> > was describing earlier.
>
> Doesn't matter, because that's not happening ;-) SIGINFO isn't
> portable, and even if it were, a signal handler couldn't possibly
> generate EXPLAIN output (remember those catalog accesses).
Well the signal gets handled by psql. It would have to be a wire protocol
message like Query Cancel that the backend watches for and handles the first
time it can.
--
greg
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2006-06-08 19:46:37 | Re: Ending EXPLAIN ANALYZE early (was Re: That EXPLAIN ANALYZE patch still needs work) |
Previous Message | Alvaro Herrera | 2006-06-08 19:35:18 | Re: [PATCHES] drop if exists remainder |