Currently atoms are being fetched one by one. There is a scald_atom_load_multiple function but it just iterates over the array of sid's and calls scald_fetch for each one. There is also an @todo tag in the function to improve it, so here is my proposal.

I added a scald_fetch_multiple, which does pretty much the same as scald_fetch, except using entity_load to load all atoms at once.

CommentFileSizeAuthor
scald-multiple_load.patch5.42 KBVolx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DeFr’s picture

Assigned: Unassigned » DeFr
Status: Needs review » Fixed

Thanks, pushed as e8575c. I've also pushed a followup commit to clean up this code, 652deb.

DeFr’s picture

Assigned: DeFr » Unassigned

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.