forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit8b49a60
committed
Cache catalog lookup data across groups in ordered-set aggregates.
The initial commit of ordered-set aggregates just did all the setup workafresh each time the aggregate function is started up. But in a GROUP BYquery, the catalog lookups need not be repeated for each group, since thecolumn datatypes and sort information won't change. When there are manysmall groups, this makes for a useful, though not huge, performanceimprovement. Per suggestion from Andrew Gierth.Profiling of these cases suggests that it might be profitable to avoidduplicate lookups within tuplesort startup as well; but changing thetuplesort APIs would have much broader impact, so I left that foranother day.1 parent92459e7 commit8b49a60
File tree
2 files changed
+290
-218
lines changed- src
- backend/utils/adt
- include/catalog
2 files changed
+290
-218
lines changed0 commit comments
Comments
(0)