Problem/Motivation
Upstream version of Radix in use by OA Radix has a performance regression. This bug has been fixed for some time in Radix but that fix has not been backported to Atrium. This reduction was originally reported here as 1 second, but the upstream report calls it 10 seconds.
Proposed resolution
Include the patch originally committed to Radix to OA Radix
Remaining tasks
Review & Commit
User interface changes
None
API changes
None
Data model changes
None
Original Report
The actual version 7.x-3.x [38eb2d99] is out of date. During some performance profiling i recognized that the radix version coming with openatrium calls drupal_get_path('module', 'field_slideshow') without checking if the module exists. This makes drupal calling file_scan_directory so radix_js_alter takes up to 1 second on my setup.
Regards,
Leo
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2552787-3-radix-performance-regression.patch | 764 bytes | Grayside |
Comments
Comment #2
Grayside commentedIn my local environment I find the impact closer to 8 seconds, for lack of the field_slideshow module. I've added an issue link where this problem was originally fixed.
Comment #3
Grayside commentedAfter discussion with @mpotter, a simple Radix update will not work. After double verifying, it seems like the specific patch in the issue I linked earlier is still compatible with the version of Radix in use by OA Radix. Here's a patch to amend the OA Radix makefile with that patch.
Comment #4
Grayside commentedComment #5
Grayside commentedComment #6
mpotter commentedCommitted to 86e2cc5.