]> granicus.if.org Git - postgresql/commit
Refactor index cost estimation functions in view of IndexClause changes.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 15 Feb 2019 18:05:19 +0000 (13:05 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 15 Feb 2019 18:05:19 +0000 (13:05 -0500)
commite89f14e2bb9f7c392c4c85a53ab5a13ea2aed83d
treebb5a4ac8c74d16bff4f4833b019b22579c6da401
parentfc6c72747ae6db6909fcd7d1adbc3d923ec1fffa
Refactor index cost estimation functions in view of IndexClause changes.

Get rid of deconstruct_indexquals() in favor of just iterating over the
IndexClause list directly.  The extra services that that function used to
provide, such as hiding clause commutation and associating the right index
column with each clause, are no longer useful given the new data structure.
I'd originally thought that it'd provide a useful amount of abstraction
by freeing callers from paying attention to the exact clause type of each
indexqual, but that hope proves to have been vain, because few callers can
ignore the semantic differences between different clause types.  Indeed,
removing it results in a net code savings, and probably some cycles shaved
by not having to build an extra list-of-structs data structure.

Also, export a few formerly-static support functions, with the goal
of allowing extension AMs to write functionality equivalent to
genericcostestimate() without pointless code duplication.

Discussion: https://postgr.es/m/24586.1550106354@sss.pgh.pa.us
contrib/bloom/blcost.c
src/backend/utils/adt/selfuncs.c
src/include/utils/selfuncs.h