Re: [PATCH] Make skipped sort explicit in EXPLAIN ANALYZE

Tom Lane <tgl@sss.pgh.pa.us>

From: Tom Lane <tgl@sss.pgh.pa.us>
To: Marti Raudsepp <marti@juffo.org>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2012-05-25T13:49:18Z
Lists: pgsql-hackers
Marti Raudsepp <marti@juffo.org> writes:
> This is just a small clarity improvement. tuplesort_performsort()
> skips sorting entirely when the result set has 0 or 1 tuples, but
> EXPLAIN still says it's using "quicksort". The patch changes that to
> "skipped"

I'm not convinced this is an improvement; it's just one more value
to confuse people with.  And any implementation of quicksort is likely
to have a fast exit path for 0 or 1 item, so I don't think the existing
display is incorrect either.

			regards, tom lane