Hi,
I think, this should be the probable fix.
There is agg_order in ParseFuncOrColumn, which should get passed on to transformAggregateCall and that should be placed in this call, instead of agg->aggorder.
Thanks,
Gokul.
On Tue, Mar 16, 2010 at 5:19 PM, Gokulakannan Somasundaram
<gokul007@gmail.com> wrote:
Hi,
I noticed a problem with the source code of 9.0Alpha 4. In parse_agg.c, there is a call made to transformSortClause.
00098 torder = transformSortClause(pstate,
00099 agg->aggorder,
00100 &tlist,
00101 true /* fix unknowns */ ,
00102 true /* force SQL99 rules */ );
00103
Here agg->aggorder should be a List of SortGroupClause pointers, whereas transformSortClause expects the second argument as a list of SortBy pointers. I verified the doxygen code by downloading the 9.0alpha4 version. I am trying to understand this piece of code, while i thought i should report this bug.
Thanks,
Gokul.
