]> granicus.if.org Git - postgresql/commit
Simplify and speed up mapping of index opfamilies to pathkeys.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 29 Nov 2010 17:29:42 +0000 (12:29 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 29 Nov 2010 17:30:43 +0000 (12:30 -0500)
commitc0b5fac7010fa3468577d3a90ee8783639500fed
tree0c49a5abb23a8dcc62790fd5d9cd9d39a210f91b
parent3c42efceb21b8dfe1b5686bd914520980933fcd4
Simplify and speed up mapping of index opfamilies to pathkeys.

Formerly we looked up the operators associated with each index (caching
them in relcache) and then the planner looked up the btree opfamily
containing such operators in order to build the btree-centric pathkey
representation that describes the index's sort order.  This is quite
pointless for btree indexes: we might as well just use the index's opfamily
information directly.  That saves syscache lookup cycles during planning,
and furthermore allows us to eliminate the relcache's caching of operators
altogether, which may help in reducing backend startup time.

I added code to plancat.c to perform the same type of double lookup
on-the-fly if it's ever faced with a non-btree amcanorder index AM.
If such a thing actually becomes interesting for production, we should
replace that logic with some more-direct method for identifying the
corresponding btree opfamily; but it's not worth spending effort on now.

There is considerably more to do pursuant to my recent proposal to get rid
of sort-operator-based representations of sort orderings, but this patch
grabs some of the low-hanging fruit.  I'll look at the remainder of that
work after the current commitfest.
src/backend/optimizer/path/indxpath.c
src/backend/optimizer/path/pathkeys.c
src/backend/optimizer/util/plancat.c
src/backend/utils/adt/selfuncs.c
src/backend/utils/cache/relcache.c
src/include/nodes/relation.h
src/include/utils/rel.h