This gets called per field, on the site I'm working on, 89 times per node when migrating 1000 nodes, so we should apply the drupal_static_fast() pattern.

CommentFileSizeAuthor
drupal_static_fast.patch928 bytescatch
XHProf: Hierarchical Profiler Report_1270101901400.png35.71 KBcatch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Active » Needs review
yched’s picture

yched CreditAttribution: yched commented
Status: Needs review » Reviewed & tested by the community

Looks good.

realityloop’s picture

realityloop
he/him
CreditAttribution: realityloop commented

drupal_static_fast.patch queued for re-testing.

marcingy’s picture

marcingy CreditAttribution: marcingy commented

drupal_static_fast.patch queued for re-testing.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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