Differential D10294

Paste - fix caching mechanism for S3-stored files
ClosedPublic
Actions

Authored by btrahan on Aug 19 2014, 6:34 PM.

Tags

None

Referenced Files

	F15580199: D10294.id24789.diff
	Wed, May 7, 4:13 AM

	F15562049: D10294.id.diff
	Wed, Apr 30, 12:07 AM

	F15559723: D10294.id24789.diff
	Tue, Apr 29, 10:13 AM

	F15559282: D10294.id24791.diff
	Tue, Apr 29, 7:59 AM

	F15557506: D10294.diff
	Mon, Apr 28, 9:55 PM

	F15507530: D10294.id24789.diff
	Tue, Apr 15, 8:50 PM

	F15495539: D10294.diff
	Sun, Apr 13, 4:51 AM

	F15490902: D10294.id24791.diff
	Fri, Apr 11, 9:35 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T5798: Fix performance issues with Paste browse list

Commits

Restricted Diffusion Commit
rPed98a1cc8464: Paste - fix caching mechanism for S3-stored files

Summary

Fixes T5798. We basically weren't using the caching mechanism. Also adds service calls for S3 stuff, and support for seeing a little info like you can for conduit.

Test Plan

uploaded a paste, looked at paste list - no s3 service calls. edited the paste, looked at paste list - no s3 service calls and edited content properly shown

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan updated this revision to Diff 24789.Aug 19 2014, 6:34 PM

btrahan added a task: T5798: Fix performance issues with Paste browse list.

btrahan retitled this revision from to Paste - fix caching mechanism for S3-stored files.

btrahan updated this object.

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

btrahan added a reviewer: epriestley.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 19 2014, 6:34 PM

Harbormaster completed remote builds in B2271: Diff 24789.Aug 19 2014, 6:35 PM

epriestley accepted this revision.Aug 19 2014, 6:57 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Aug 19 2014, 6:57 PM

Closed by commit rPed98a1cc8464 (authored by @btrahan).

Revision Contents
Changeset List

Path

Size

src/

aphront/

console/

plugin/

DarkConsoleServicesPlugin.php

1 line

applications/

files/

engine/

PhabricatorS3FileStorageEngine.php

27 lines

paste/

query/

PhabricatorPasteSearchEngine.php

4 lines

Diff 24791

src/aphront/console/plugin/DarkConsoleServicesPlugin.php

Loading...

src/applications/files/engine/PhabricatorS3FileStorageEngine.php

Loading...

src/applications/paste/query/PhabricatorPasteSearchEngine.php

Loading...