Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
\Drupal\facets\Plugin\facets\url_processor\QueryString::buildUrls performance should be improved.
For bigger results sets site has long response times.
And with increasing amount of facet's results it becomes bigger and bigger.
Comment | File | Size | Author |
---|---|---|---|
#2 | query_string_build_urls-2676210-2.patch | 1.28 KB | Evaldas Užkuras |
Comments
Comment #2
Evaldas UžkurasThe problem, is that request and url from request generation is used for all result's items instead of one time per function call.
Comment #3
borisson_Sure, this change looks great and the tests still pass. This makes sense.
Comment #4
borisson_Thanks for this optimisation, committed!
Comment #7
Shyamala CreditAttribution: Shyamala at UniMity Solutions Pvt Limited commentedEven with this patch applied,
we find the facet block causes 1/4 of the queries on the searchpage. The facet also shows as a slow code on New relic.
Planning to load facet block using Ajax.
Any other recommendations to optimize same?
Comment #8
borisson_Would you mind opening a new issue instead? This patch has been committed for over 2 years. There is no other clear path for optimisation, if there was - we'd execute on it quickly :)