Page MenuHomePhabricator

Make logic for streaming VCS stuff directly to Files more reusable
ClosedPublic

Authored by epriestley on Aug 26 2016, 3:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 9:30 AM
Unknown Object (File)
Tue, Apr 9, 2:21 AM
Unknown Object (File)
Wed, Apr 3, 9:13 AM
Unknown Object (File)
Thu, Mar 28, 10:19 PM
Unknown Object (File)
Thu, Mar 28, 9:39 AM
Unknown Object (File)
Thu, Mar 28, 4:51 AM
Unknown Object (File)
Tue, Mar 26, 6:28 PM
Unknown Object (File)
Tue, Mar 26, 5:23 PM
Subscribers
None

Details

Summary

Ref T11524. Ref T10423. Earlier, I converted diffusion.filecontentquery to put the actual file content in Files, then return a PHID for the file, instead of trying to send the content over Conduit.

In T11524, we have a similar set of problems with diffs that contain non-UTF8 data (and, in T10423, diffs that are simply enormous).

I want to provide an API method to do the same sort of thing with diff output (like from git diff), so we call the method, it shoves the data in Files, and then we go pull it out of Files.

To support this, take the "shove the output of a Future into Files" logic and put it in a new base FileFuture query. This will let me make RawDiffQuery share the logic more easily.

Test Plan

Browsed Diffusion, ran diffusion.filecontentquery to fetch file content.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make logic for streaming VCS stuff directly to Files more reusable.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 26 2016, 4:02 PM
This revision was automatically updated to reflect the committed changes.