-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Http.put() should stream the content #651
Comments
@emilianodellacasa can you take a look? |
@yegor256 Sure, please assign these to me |
@0crat assign @emilianodellacasa |
@emilianodellacasa/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please. |
@0crat wait - going on vacation for a few days |
@emilianodellacasa The impediment for #651 was registered successfully by @emilianodellacasa/z |
@0crat assign @v-kolesnikov |
See the changes made in #650.
Http.get_file()
now reads the content from HTTP page and saves it to the file, without touching the memory (well, only in small chunks). We should do the same forHttp.put()
. At the moment we are usingIO.read()
in order to read the entire file into memory first. This is not effective. Let's stream.The text was updated successfully, but these errors were encountered: