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

	F14417118: D10294.diff
	Wed, Dec 25, 12:10 AM

	Unknown Object (File)
	Tue, Dec 10, 4:02 PM

	Unknown Object (File)
	Thu, Dec 5, 1:45 AM

	Unknown Object (File)
	Tue, Dec 3, 12:27 AM

	Unknown Object (File)
	Sat, Nov 30, 2:36 AM

	Unknown Object (File)
	Sat, Nov 30, 2:36 AM

	Unknown Object (File)
	Wed, Nov 27, 3:07 PM

	Unknown Object (File)
	Nov 23 2024, 8:46 AM

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...