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.
I am playing with D8 and facets, and noticed that in a clean installation, around 2gb ram, installing facets_range_widget the server struggles a lot. First I have to increase memory in php.ini, then increase the max_time, and finally a time ERR_EMPTY_RESPONSE is shown in the browser.
Simply doing:
drush8 pm-uninstall -y facets_range_widget
"Fixes" the issue, so I assume it is a problem in the module.
Comment | File | Size | Author |
---|---|---|---|
#16 | add_testcoverage_for-2894637-16.patch | 17.91 KB | borisson_ |
Comments
Comment #2
alexmoreno CreditAttribution: alexmoreno commentedComment #3
borisson_We've noticed before that ranges with the database backend can be really slow, and as long as we're not yet implementing a solr-specific query it'll be really slow for solr as well. I'd like to know with how many results you're seeing this slowdown. That might help us figure out a way to write a test for this.
Comment #4
alexmoreno CreditAttribution: alexmoreno commentedI'm actually using solr, and is a clean db, so not many results). Just 50 nodes for the whole site db created with devel generate, so results can be like 10 or so?
Comment #5
borisson_Yeah, that shouldn't be too much for the intended usecase. I've been trying to figure out how to measure this in a test. No clue if this is a good path forward.
Comment #6
borisson_Looks like the processor didn't have any specific testcoverage yet. I've tried to increase the amount of results to make this take longer but I can't seem to make it take > 1 second.
Comment #7
borisson_Comment #8
borisson_We should improve the tests to figure out why this is so slow and see if we can reproduce the issue in the test so we know how a solid way to resolve the actual issue.
Needs work for that.
Comment #9
borisson_I figured that maybe it was in the widget, but I can't seem to find it.
I think I found a bug though by writing explicit test coverage.
Comment #10
borisson_Looks like the interdiff in #9 was for a different issue.
Comment #12
borisson_Crediting agoradesign for review in #2872404: Use native facet range queries with solr + the patch there, that I merged into this patch. This also adds more tests.
Comment #13
borisson_Comment #14
borisson_Rescoping this issue, created a followup for the actual performance issues.
Comment #15
borisson_Comment #16
borisson_17kb of mostly test-coverage, this makes me very happy!
Comment #17
borisson_Committed and pushed, followup in #2872404: Use native facet range queries with solr