Homec4science

Put a cache in front of Celerity transforms, and update packages

Authored by epriestley <git@epriestley.com> on May 22 2014, 19:47.

Description

Put a cache in front of Celerity transforms, and update packages

Summary:
Fixes T5094. In some cases we do slightly expensive transformations to resources (inlining images, replacing URIs, building packages). We can throw cache in front of them easily since URIs are already permanently associated with a single resource.

Also browse around and move some CSS/JS into packages.

Test Plan:
Added logging to verify the caches are working, saw moderately improved performance.

Browsed around looking at resources tab in developer console, saw fewer total requests.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T5094

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

Details

Committed
epriestley <git@epriestley.com>May 22 2014, 19:47
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf1534e6feb6a: Make password reset emails use one-time tokens
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHf896dc5392a0: Put a cache in front of Celerity transforms, and update packages (authored by epriestley <git@epriestley.com>).May 22 2014, 19:47