Differential D13480

Cache remarkup engines in Feed
ClosedPublic
Actions

Authored by epriestley on Jun 29 2015, 10:18 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 14, 9:46 PM

	Unknown Object (File)
	Thu, Apr 11, 9:36 AM

	Unknown Object (File)
	Tue, Apr 2, 3:36 AM

	Unknown Object (File)
	Mon, Apr 1, 3:29 PM

	Unknown Object (File)
	Mon, Apr 1, 6:45 AM

	Unknown Object (File)
	Fri, Mar 29, 11:44 PM

	Unknown Object (File)
	Thu, Mar 28, 7:05 AM

	Unknown Object (File)
	Tue, Mar 26, 12:13 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T8631: Straighten out the fetch plan for feed stories

Commits

Restricted Diffusion Commit
rPa8f5e10b790f: Cache remarkup engines in Feed

Summary

Ref T8631. This method uses cached settings.

Test Plan

Saw 40% performance improvement locally (reduced 200ms of 500ms).

Diff Detail

Repository

Branch

perf3

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7055
Build 7163: [Placeholder Plan] Wait for 30 Seconds
Build 7162: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 32616.Jun 29 2015, 10:18 PM

epriestley retitled this revision from to Cache remarkup engines in Feed.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T8631: Straighten out the fetch plan for feed stories.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 29 2015, 10:18 PM

btrahan accepted this revision.Jun 30 2015, 6:12 PM

btrahan edited edge metadata.

btrahan added inline comments.

src/applications/feed/story/PhabricatorFeedStory.php
523	Looks like we have this in a few more spots: https://secure.phabricator.com/diffusion/P/browse/master/?grep=PhabricatorMarkupEngine%3A%3AnewMarkupEngine

This revision is now accepted and ready to land.Jun 30 2015, 6:12 PM

Closed by commit rPa8f5e10b790f: Cache remarkup engines in Feed (authored by epriestley, committed by epriestley). · Explain WhyJun 30 2015, 6:20 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

feed/

story/

PhabricatorFeedStory.php

2 lines

Diff 32616

src/applications/feed/story/PhabricatorFeedStory.php

Loading...