It appears that around 10 - 15 SQL queries are executed per page just to get the facet settings. There should really only be one per searcher. Lets liik into the FacetapiAdapter::getFacetSettings*() methods to see if we can make them more efficient.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | efficient-settings-1394204-3.patch | 2.52 KB | cpliakas |
| #2 | efficient-settings-1394204-2.patch | 4 KB | cpliakas |
Comments
Comment #1
cpliakas commentedChanging to a performance bug.
Comment #2
cpliakas commentedThe attached patch adds a function that loads all settings associated with a searcher and stored them in a static. That way other functions and methods can leverage this function to get cached settings that prevent multiple calls to the database.
Comment #3
cpliakas commentedChanged function and method names for transparency.
Comment #4
cpliakas commentedCommitted at http://drupalcode.org/project/facetapi.git/commit/634817c.
Comment #5
cpliakas commentedAdding "Performance" tag.
Comment #6.0
(not verified) commentedUpdated issue summary.