Re: Fix for select sum(2+2)... - Mailing list pgsql-hackers

From Vadim B. Mikheev
Subject Re: Fix for select sum(2+2)...
Date
Msg-id 34BD8920.4C4BC803@sable.krasnoyarsk.su
Whole thread Raw
In response to Re: Fix for select sum(2+2)...  (Bruce Momjian <maillist@candle.pha.pa.us>)
List pgsql-hackers
Bruce Momjian wrote:
>
> I have decided the whole qry_aggs is bad.  It is bad becuase it makes
> multiple copies of Aggreg, and both copies must be processed by any
> changes by rewrite and optimizer.  I am removing the field
> Query->qry_aggs, and replacing it with a function that will called when
> creating the Agg which spins through the Plan target list and returns a
> linked list of Agg*.  Much cleaner, and I can remove much of the special
> qry_aggs handling I added to get other Agg stuff to work.
>
> Also, this relates to the SubLink change.  I am now going to recommend
> not having a separate Sublink pointer list field in Query, but adding a
> function that will return a list of valid Sublink entries in from
> qry->qual, or maybe even qry->targetlist.

Agreed. This will also simplify readfuncs.c

Vadim

pgsql-hackers by date:

Previous
From: "Vadim B. Mikheev"
Date:
Subject: Re: [QUESTIONS] Arrays (inserting and removing)
Next
From: "Thomas G. Lockhart"
Date:
Subject: Re: [HACKERS] Re: [QUESTIONS] trouble grouping rows