aboutsummaryrefslogtreecommitdiff
path: root/src/backend/parser/parse_utilcmd.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2011-03-22 00:34:31 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2011-03-22 00:34:31 -0400
commit8df08c84894001d3d3f5d10b3290a1063a453316 (patch)
tree138335fc92bf63822f93fd68203b1c97fa4db3c6 /src/backend/parser/parse_utilcmd.c
parent6d8096e2f3f2c1296fa880f44f3fa5701b2f40c4 (diff)
downloadpostgresql-8df08c84894001d3d3f5d10b3290a1063a453316.tar.gz
postgresql-8df08c84894001d3d3f5d10b3290a1063a453316.zip
Reimplement planner's handling of MIN/MAX aggregate optimization (again).
Instead of playing cute games with pathkeys, just build a direct representation of the intended sub-select, and feed it through query_planner to get a Path for the index access. This is a bit slower than 9.1's previous method, since we'll duplicate most of the overhead of query_planner; but since the whole optimization only applies to rather simple single-table queries, that probably won't be much of a problem in practice. The advantage is that we get to do the right thing when there's a partial index that needs the implicit IS NOT NULL clause to be usable. Also, although this makes planagg.c be a bit more closely tied to the ordering of operations in grouping_planner, we can get rid of some coupling to lower-level parts of the planner. Per complaint from Marti Raudsepp.
Diffstat (limited to 'src/backend/parser/parse_utilcmd.c')
0 files changed, 0 insertions, 0 deletions