From: | Mark Kirkwood <markir(at)paradise(dot)net(dot)nz> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Neil Conway <neilc(at)samurai(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>, "eng(at)intranet(dot)greenplum(dot)com" <eng(at)intranet(dot)greenplum(dot)com> |
Subject: | Re: Avg performance for int8/numeric |
Date: | 2007-02-17 00:13:51 |
Message-ID: | 45D648BF.4020607@paradise.net.nz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Bruce Momjian wrote:
> I have tested this patch but it generates regression failures.
>
> There was some code drift, so I am attaching an updated version of the
> patch, and the regression diffs. The 'four' column is an 'int4' so my
> guess is that somehow the wrong aggregate is being called.
>
Good catch - I must have neglected to run the regression test after
amending the number of array arguments for the numeric avg :-(.
Hmmm - this changing the number of array args for avg means we can't mix
transition functions for variance with final functions for avg - which
is exactly what the regression suite does with the 'newavg' aggregate.
I've 'fixed' this by amending the definition of 'newavg' to use the
transition and final function that 'avg' does. However I found myself
asking if this lost us the point of that test - so I looked back at the
older postgres versions (e.g. 7.1.3) and saw that back *then* 'newavg'
and 'avg' were defined using the same functions...so I think making the
change as indicated is ok.
I've attached a new patch with this change.
Cheers
Mark
Attachment | Content-Type | Size |
---|---|---|
avg3.patch | text/x-patch | 8.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tatsuo Ishii | 2007-02-17 00:23:16 | n-gram search function |
Previous Message | Tom Lane | 2007-02-16 23:53:05 | Re: autovacuum next steps |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2007-02-17 00:23:41 | Re: Avg performance for int8/numeric |
Previous Message | Brendan Jurd | 2007-02-17 00:13:37 | Re: [GENERAL] ISO week dates |