Homec4science

Cache viewer spaces

Authored by epriestley <git@epriestley.com> on Jun 17 2015, 19:43.

Description

Cache viewer spaces

Summary:
Ref T8575. Although we cache spaces as a whole, we don't cache viewer spaces. This can still do a lot of work if they have complex policies.

Instead, cache them in the request cache.

Test Plan: Saw this account for 37% of a page in produciton (303ms on on 835ms).

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T8575

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

Details

Committed
epriestley <git@epriestley.com>Jun 17 2015, 19:43
Pushed
aubortJan 31 2017, 17:16
Parents
rPHaf8ad58e6bb1: Probably fix notifications not being properly marked as unread
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH984976ce2029: Cache viewer spaces (authored by epriestley <git@epriestley.com>).Jun 17 2015, 19:43