高增琦 <pgf00a@gmail.com> writes:
> In transformDeleteStmt:
> qry->hasWindowFuncs = pstate->p_hasWindowFuncs;
> qry->hasAggs = pstate->p_hasAggs;
> if (pstate->p_hasAggs)
> parseCheckAggregates(pstate, qry);
> Do we support using agg or window function in delete statement?
> Or, this code should be removed?
I think it's dead code given the syntactic limitations on DELETE,
but I would not be in favor of removing it. Better to have it there to
keep transformDeleteStmt looking as much as possible like the other ones.
It's not like that's either expensive or a lot of code.
An example of why this would be penny-wise and pound-foolish is that
we might choose to apply the check that you can't write aggregates in
DELETE inside parseCheckAggregates. (We don't, but it's not an impossible
future restructuring.)
regards, tom lane
