aboutsummaryrefslogtreecommitdiff
path: root/src/include/postgres.h
diff options
context:
space:
mode:
authorDavid Rowley <drowley@postgresql.org>2021-07-04 18:47:31 +1200
committerDavid Rowley <drowley@postgresql.org>2021-07-04 18:47:31 +1200
commit63b1af94375cc2be06a5d6a932db24cd8e9f45e9 (patch)
treeeb3a09111a707b132221674895b9190ee06bab4c /src/include/postgres.h
parent792259591c0fc19c42247fc7668b1064d1e850d4 (diff)
downloadpostgresql-63b1af94375cc2be06a5d6a932db24cd8e9f45e9.tar.gz
postgresql-63b1af94375cc2be06a5d6a932db24cd8e9f45e9.zip
Cleanup some aggregate code in the executor
Here we alter the code that calls build_pertrans_for_aggref() so that the function no longer needs to special-case whether it's dealing with an aggtransfn or an aggcombinefn. This allows us to reuse the build_aggregate_transfn_expr() function and just get rid of the build_aggregate_combinefn_expr() completely. All of the special case code that was in build_pertrans_for_aggref() has been moved up to the calling functions. This saves about a dozen lines of code in nodeAgg.c and a few dozen more in parse_agg.c Also, rename a few variables in nodeAgg.c to try to make it more clear that we're working with either a aggtransfn or an aggcombinefn. Some of the old names would have you believe that we were always working with an aggtransfn. Discussion: https://postgr.es/m/CAApHDvptMQ9FmF0D67zC_w88yVnoNVR2+kkOQGUrCmdxWxLULQ@mail.gmail.com
Diffstat (limited to 'src/include/postgres.h')
0 files changed, 0 insertions, 0 deletions