codeintel: GetUpload should return reader, not file

Created by: efritz

This saves us from writing a temporary file on disk, and also saves us from waiting for a very large file to transfer before starting processing.

Returning a reader seems more idiomatic, and allows us to begin processing the data as it is streamed in from the bundle manager.