The scald_types() function performs a database query when it is called. It would be a good idea to statically cache this result since this function is called often.

CommentFileSizeAuthor
#2 2291759-2-scald-scald_types_static_cache.patch1.86 KBpfrenssen

Comments

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Issue summary: View changes
pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Active » Needs review
StatusFileSize
new 2291759-2-scald-scald_types_static_cache.patch1.86 KB
jcisio’s picture

jcisio commented
Status: Needs review » Fixed

I've changed the cache reset to keep it parity-wise with other functions and to keep static cache implementation internal to scald_types(). Committed and pushed. Thanks.

  • Commit 6af7202 on 7.x-1.x authored by pfrenssen, committed by jcisio: 
    Issue #2291759 by pfrenssen: Added Provide static caching for...

Status: Fixed » Closed (fixed)

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