Differential D13480

Cache remarkup engines in Feed
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F14058614: D13480.id32616.diff
	Sun, Nov 17, 1:20 PM

	F13981685: D13480.id32635.diff
	Oct 19 2024, 6:27 PM

	F13977235: D13480.id.diff
	Oct 18 2024, 5:44 PM

	F13973447: D13480.diff
	Oct 18 2024, 12:56 AM

	F13973429: D13480.id.diff
	Oct 18 2024, 12:53 AM

	F13973423: D13480.id32616.diff
	Oct 18 2024, 12:52 AM

	F13973420: D13480.id32635.diff
	Oct 18 2024, 12:51 AM

	F13973102: D13480.diff
	Oct 17 2024, 11:08 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...