Homec4science

Allow diffusion.filecontentquery to load data for arbitrarily large files

Authored by epriestley <git@epriestley.com> on Jan 21 2016, 13:25.

Description

Allow diffusion.filecontentquery to load data for arbitrarily large files

Summary:
Fixes T10186. After D14970, diffusion.filecontentquery puts the content in a file and returns the file PHID.

However, it does this in a way that doesn't go through the chunking engine, so it will fail for files larger than the chunk threshold (generally, 8MB).

Instead, stream the file from the underlying command directly into chunked storage.

Test Plan:

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10186

Differential Revision: https://secure.phabricator.com/D15072

Details

Committed
epriestley <git@epriestley.com>Jan 21 2016, 18:52
Pushed
aubortJan 31 2017, 17:16
Parents
rPHe0fef74be782: Minor CSS tweaks to autocomplete
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHb51a859636ac: Allow diffusion.filecontentquery to load data for arbitrarily large files (authored by epriestley <git@epriestley.com>).Jan 21 2016, 18:52