Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches
It will be applied as soon as one of the PostgreSQL committers reviews and approves it. --------------------------------------------------------------------------- Mark Kirkwood wrote: > Neil Conway wrote: > > On Fri, 2006-11-24 at 11:08 +1300, Mark Kirkwood wrote: > >> - Modifies do_numeric_accum to have an extra bool parameter and does not > >> calc sumX2 when it is false. > > > > I think it would be clearer to reorganize this function slightly, and > > have only a single branch on "useSumX2". On first glance it isn't > > obviously that transdatums[2] is defined (but unchanged) when useSumX2 > > is false. > > > > Right - new patch attached that adds a new function do_numeric_avg_accum > that only uses N and sum(X). This means I could amend the avg aggregates > for numeric, int8 to have a initvalues of {0,0}. > > Cheers > > Mark > > ---------------------------(end of broadcast)--------------------------- > TIP 2: Don't 'kill -9' the postmaster -- Bruce Momjian <[EMAIL PROTECTED]> http://momjian.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 7: You can help support the PostgreSQL project by donating at http://www.postgresql.org/about/donate