Vallimaharajan G <vallimaharajan.gs@zohocorp.com> writes:
> I am currently referring to the Postgres source code (psql (PostgreSQL) 14.3) and came across a particular section
relatedto window aggregate initialization that has left me with a question. Specifically, I noticed a conditional case
inthe initialization of per aggregate (initialize_peraggregate in nodeWindowAgg.c) where the winstate frameOptions is
beingchecked, but it appears that frameOptions is not set before this conditional case.
You are right, and that's a bug. It's not of major consequence ---
it would just cause us to set up for moving-aggregate mode when
we won't ever actually move the frame head -- but it's at least a
little bit inefficient.
While I'm looking at it, there's a pretty obvious typo in the
adjacent comment:
- * and collect the right set of fields from the pg_attribute entry.
+ * and collect the right set of fields from the pg_aggregate entry.
regards, tom lane