Homec4science

Reduce the cost of loading large numbers of macros

Authored by epriestley <git@epriestley.com> on Sep 6 2014, 02:30.

Description

Reduce the cost of loading large numbers of macros

Summary:
Ref T6013. I accidentally made this cost explosviely huge when fixing macros for logged out users in D10411.

Specifically, we'd load all the macros, which would load all the files, which would load all the macros (to do policy checks), which would fill out of cache I think (but maybe only some of the time?). Anyway, bad news.

Instead, only load the files if we need them.

Test Plan: Viewed macro main page, macro detail, used a macro, used a meme, edited a macro, edited audio.

Reviewers: btrahan, csilvers

Reviewed By: csilvers

Subscribers: epriestley, spicyj

Maniphest Tasks: T6013

Differential Revision: https://secure.phabricator.com/D10428

Details

Committed
epriestley <git@epriestley.com>Sep 6 2014, 02:30
Pushed
aubortJan 31 2017, 17:16
Parents
rPHac4247ea59eb: Provide more information from diffusion.querycommits
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHb772a2b92af5: Reduce the cost of loading large numbers of macros (authored by epriestley <git@epriestley.com>).Sep 6 2014, 02:30