Re: Our CLUSTER implementation is pessimal - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Our CLUSTER implementation is pessimal
Date
Msg-id 12948.1220878066@sss.pgh.pa.us
Whole thread Raw
In response to Re: Our CLUSTER implementation is pessimal  (Gregory Stark <stark@enterprisedb.com>)
Responses Re: Our CLUSTER implementation is pessimal  (Gregory Stark <stark@enterprisedb.com>)
List pgsql-hackers
Gregory Stark <stark@enterprisedb.com> writes:
> Yeah, I've been thinking about how to use the planner to do this.

I thought the answer to that was going to be more or less "call
cost_sort() and cost_index() and compare the answers".

> To do that it seems to me what we would need to do is add a function
> _pg_get_rawtuple_header() which returns the visibility information that HTSV
> needs. 

You seem to be confusing "use the planner" with "use the executor".
All that we need here is a decision about which code path to take
within CLUSTER.  We don't need to bring in boatloads of irrelevant
infrastructure --- especially not infrastructure that's going to be
fighting us every step of the way.  The executor isn't designed to
return raw tuples and no magic function is going to change that.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCH] Cleanup of GUC units code
Next
From: Gregory Stark
Date:
Subject: Re: [PATCH] Cleanup of GUC units code