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.While debugging some pretty paths issue, I noticed the url_processor (being a pre query processor) is initialized twice.
First in Facet.php (loadProcessors), later in the DefaultFacetManager.php.
We can probably skip creating the second instance in the facet manager and using loadProcessors on the given Facet instead.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2933799-2--do-not-reinitialize-prequeryprocessors.patch | 1.16 KB | StryKaizer |
Comments
Comment #2
StryKaizerDid not ran test locally, lets see what this does.
Comment #3
borisson_Awesome, this is another performance improvement. Tagging as such to make sure we can document those in the next release. I'd like to do some tests to test the actual improvement, so we have numbers. Just testing this with ab should be sufficient.
Comment #5
borisson_I tested this with ab, and there is a significant improvement. Great work @StryKaizer!