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
Large memory consumption #105
Comments
I guess problem is located here: Line 76 in 4216ac8
You shouldn't read whole file into memory when you have file streams and binary reads :(. |
Looks like we can likely use https://hexdocs.pm/ex_aws/1.0.0-beta1/ExAws.S3.html#upload_part/6 to upload in chunks. |
Can you try the It has some quirks with error handling, but a large file upload should work better. |
Hi. Something wrong with ex_aws library, yes? |
Same error here. Also trying to get large uploads working. Looking into the code. |
The latest ExAws wasn't ready yet for file streaming when I tried. If you want to try updating that branch and see if it helps, I would appreciate it! |
Hey @stavro. As far as I tested this, it worked great with the current version of
|
Amazing. Should this library then require sweet_xml? I thought AWS had a way of requesting errors in JSON. If it's possible to request errors in JSON we should totally push that down to ExAws! |
Good one. It shouldn't be too involving to change that, as the parser in ex_aws is just a module, it should be easy to create a json parser. Anyway this works for now :) Probably best to include sweet_xml in this library indeed for using it with ex_aws 1.0. |
To confirm, no trouble in uploading 500+mb files on a small heroku dyno. |
Amazing! Thanks for looking into it. I'll update the ex_aws_beta branch and start adding documentation about it next week. If you run into any other quirks please let me know. As soon as ExAws is out of beta we'll merge it into here. 🎉 |
Cool, I run it in production, because we need to support large file sizes. But it's ok because we have tests, and it seems to work fine 💃 Will report any issues that come up. |
We will use ex_aws_beta brunch in production, so It would be very awesome if you will notify us here when you will release next version of hex package, so we will stop using github repo dependency :). |
Same here, it might be good to publish a beta release? |
Will do on Monday or Tuesday. Thanks for helping test everyone! On Sep 30, 2016 11:50 PM, "Jaap Frolich" notifications@github.com wrote:
|
Released arc as v0.6.0-rc1 to track ExAws. Please try and report any feedback. Thanks! |
Right now BEAM process seems to require space of about 3x file size. Maybe you can suggest how to send file directly to S3 without reading it into memory? We deal with very large files (up to 2 Gb).
The text was updated successfully, but these errors were encountered: