Re: [PATCHES] Proof of concept for MIN/MAX optimization

2005-04-14 Thread Mark Kirkwood
Tom Lane wrote: Mark Kirkwood <[EMAIL PROTECTED]> writes: The handling of nulls is a little unexpected (still todo?) : Yeah, that was broken in the first draft :-( ... I think it's OK in the committed version though. (post cvs update) yeah - looks good! regards Mark ---(end

Re: [PATCHES] Proof of concept for MIN/MAX optimization

2005-04-14 Thread Tom Lane
Mark Kirkwood <[EMAIL PROTECTED]> writes: > The handling of nulls is a little unexpected (still todo?) : Yeah, that was broken in the first draft :-( ... I think it's OK in the committed version though. regards, tom lane ---(end of broadcast)--

Re: [PATCHES] Proof of concept for MIN/MAX optimization

2005-04-13 Thread Mark Kirkwood
Tom Lane wrote: I haven't done the work yet to add a column to pg_aggregate, so this just knows about optimizing a couple of hard-wired cases (int4 and text). Other than that it's reasonably complete, I think. Comments? Looks good : regression=# explain select max(unique1) from tenk1;

[PATCHES] Proof of concept for MIN/MAX optimization

2005-04-10 Thread Tom Lane
I haven't done the work yet to add a column to pg_aggregate, so this just knows about optimizing a couple of hard-wired cases (int4 and text). Other than that it's reasonably complete, I think. Comments? regards, tom lane binNkvLb5n8b6.bin Description: minmax-0.patch.g