2009/11/19 Hitoshi Harada <umi.tanuki@gmail.com>:
> 2009/11/19 Andrew Gierth <andrew@tao11.riddles.org.uk>:
>> Small nitpicks: there are some comments not updated (e.g. the
>> big one at the start of eval_windowaggregates). A couple of lines are
>> commented-out using C++ comments.
Fixed. Document patch is included as well.
>> - missing _readWindowFrameDef function (all nodes that are output
>> from parse analysis must have both _read and _out functions,
>> otherwise views can't work)
I changed my mind and WindowFrameDef is alive only in initial parser
stage so that _readWindowFrameDef is now unnecessary. Information of
startOffset and endOffset will be copied to WindowClause members.
>> - the A_Const nodes should probably be transformed to Const nodes in
>> parse analysis, since A_Const has no _read/_out functions, which
>> means changing the corresponding code in the executor.
Fixed.
>> - ruleutils.c not updated to deparse the newly added window options
Fixed.
>> - leaks memory like it's going out of style
As earlier mail, I added aggcontext to WindowAggState.
Still it doesn't contain RANGE ... PRECEDING / FOLLOWING. If it's not
acceptable for commit without RANGE value support, I'd agree with
that. I'm planning to work on that until the next CommitFest.
Regards,
--
Hitoshi Harada
