Bug 498309

Cause: QN-ReQ

  • Evidence: "This is slow for a number of reasons - not only do we have to call a template, but get_text uses messages.html.tmpl, which loads field-descs.none.tmpl, which calls into the DB to get the fields, and then loops over each field. For each comment"

Fix: BQ

  • "There are a number of issues with this, but the simplest fix is to cache the term (per language, even though in this case its not needed"

Interesting Finding:

  • Developers may not prefer complex fixes:
    • "Yeah, this is definitely too complex. Let's keep it in Bugzilla::Util and just cache the returned results somewhere in request_cache?"