To: "Mark Kirkwood" <markir@paradise.net.nz>
Cc: "Luke Lonergan" <llonergan@greenplum.com>,
	"Heikki Linnakangas" <heikki@enterprisedb.com>,
	"Anton" <anton200@gmail.com>,  <pgsql-performance@postgresql.org>
Subject: Re: partitioned table and ORDER BY indexed_field DESC LIMIT 1
In-Reply-To: <4726C52E.5020909@paradise.net.nz> (Mark Kirkwood's message of
	"Tue\, 30 Oct 2007 18\:46\:22 +1300")
User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.1 (gnu/linux)
References: <C34C09CA.43C59%llonergan@greenplum.com>
	<4726C52E.5020909@paradise.net.nz>
From: Gregory Stark <stark@enterprisedb.com>
Organization: EnterpriseDB
Date: Tue, 30 Oct 2007 09:06:21 +0000
Message-ID: <87zly1as4y.fsf@oxford.xeocode.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

"Mark Kirkwood" <markir@paradise.net.nz> writes:

> Luke Lonergan wrote:
>> Sure - it's here:
>>   http://momjian.us/mhonarc/patches_hold/msg00381.html
>
> To clarify - we've fixed this in Greenplum db - the patch as submitted is
> (hopefully) a hint about how to fix it in Postgres, rather than a working
> patch... as its full of non-postgres functions and macros:

Oh, that was the problem with the original patch and I thought Luke had said
that was the problem which was fixed.

> cdbpathlocus_pull_above_projection

In particular this is the function I was hoping to see. Anyways as Tom pointed
out previously there's precedent in Postgres as well for subqueries so I'm
sure I'll be able to do it.

(But I'm still not entirely convinced putting the append member vars into the
eclasses would be wrong btw...)

-- 
  Gregory Stark
  EnterpriseDB          http://www.enterprisedb.com
  Ask me about EnterpriseDB's On-Demand Production Tuning