Fabien COELHO wrote:
> The answer is essentially yes, the field is needed for the "aggregated" mode
> where this specific behavior is used.
OK, thanks, that looks better to me.
Can you now appreciate why I asked for split patches? If I had to go
over the big patch I probably wouldn't have been able to read through
each to make sense of it.
I pushed this, along with a few more tweaks, mostly adding comments and
moving functions so that related things are together. I hope I didn't
break anything.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services