Re: array_agg (was Re: [HACKERS] The Axe list)

2008-11-04 Thread Peter Eisentraut
Ian Caulfield wrote: 2008/10/15 Ian Caulfield [EMAIL PROTECTED]: I started to look at implementing array_agg by making the existing intagg stuff more generic ... and here's what I've come up with. Going through the commit fest listings, I think we can safely omit this patch and work out an

array_agg (was Re: [HACKERS] The Axe list)

2008-10-15 Thread Ian Caulfield
2008/10/15 Ian Caulfield [EMAIL PROTECTED]: I started to look at implementing array_agg by making the existing intagg stuff more generic ... and here's what I've come up with. I've currently implemented this as a contrib module to make it quicker to develop/test. The aggregate uses the same

Re: array_agg (was Re: [HACKERS] The Axe list)

2008-10-15 Thread Robert Haas
I've been taking a look at this as well and came up with a slightly different approach. The attached patch is intended to go in core (not contrib) and uses some array-construction facilities that already exist in core. I'm not sure which approach is better, so I'll throw this out there with