- Issue created by @deepakkm
- 🇮🇳India deepakkm
Adding screenshot for single and multiple asset. I do see it just gives around 7-8 assets in result but takes around 2 sec which is still higher in number
- First commit to issue fork.
- 🇮🇳India rajeshreeputra Pune
Most probably issue introduced because of the AvailabilityPublishingStatusCompare.php.
The AvailabilityPublishingStatusCompare field is calling
$this->assetUpdateChecker->checkAssets($values->_entity)
for every single row in the media library view during the render() method.The Problem
N+1 Performance Issue:
- Media library loads: 12 assets per page
- Each asset triggers:
checkAssets()
call in render() - Result: 12 individual API calls to Acquia DAM
Why this happens
- The
render()
method is called for every row when the view is displayed.
Solution
- Remove the "Availability/Publishing status" field from Media library.
- First commit to issue fork.
- Merge request !163Fix Media Library takes around 8-10 seconds to load. → (Merged) created by chandu7929
- First commit to issue fork.
-
japerry →
committed ba3d4697 on 1.1.x authored by
chandu7929 →
Issue #3526555: Media Library takes around 8-10 seconds to load
-
japerry →
committed ba3d4697 on 1.1.x authored by
chandu7929 →
- 🇺🇸United States japerry KVUO
Huh interesting find! Indeed adding the version/external id speeds up the call. I presume since its otherwise trying to fetch them. Fixed.
Automatically closed - issue fixed for 2 weeks with no activity.