Page MenuHomePhabricator

Convert user profile images into a standard cache
ClosedPublic

Authored by epriestley on Jun 5 2016, 1:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 5, 8:03 AM
Unknown Object (File)
Wed, Dec 4, 10:16 AM
Unknown Object (File)
Thu, Nov 28, 2:34 AM
Unknown Object (File)
Tue, Nov 19, 4:10 PM
Unknown Object (File)
Sat, Nov 16, 2:54 PM
Unknown Object (File)
Tue, Nov 12, 5:51 AM
Unknown Object (File)
Nov 7 2024, 8:53 PM
Unknown Object (File)
Oct 31 2024, 11:34 AM
Subscribers
None

Details

Summary

Ref T4103. Ref T10078. This moves profile image caches to new usercache infrastructure.

These dirty automatically based on configuration and User properties, so add some stuff to make that happen.

This reduces the number of queries issued on every page by 1.

Test Plan

Browsed around, changed profile image, viewed as self, viewed as another user, verified no more query to pull this information on every page

Diff Detail

Repository
rP Phabricator
Branch
setting30
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12466
Build 15798: Run Core Tests
Build 15797: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Convert user profile images into a standard cache.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Those failures are probably fixed in D16041, let me backport the fixes.

Oh, no, they're not directly related. Some unit tests try to build/access users in weird ways that don't work with some of the cache stuff. I'll sort it out in the morning.

epriestley edited edge metadata.
  • Rebase on D16042, which fixes unit tests.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 5 2016, 2:53 PM