-
-
Notifications
You must be signed in to change notification settings - Fork 439
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
Better large file handling #3
Comments
Define large? Just tried a 600MB ISO with no problem. It does seem to assume that all UDP ports are open to the internet, it seems to try a random port each time. |
I made some changes and this is no longer an issue! I'd still like performance to be better, but this issue can be closed! |
Actually, this is still broken. I get an error when actually trying to download the file. Something like "Error: No File" (in chrome). However, the data is being transferred, so it's an error creating the final blob and url. Will investigate soon. |
You should see how FileSaver.js handled this: https://github.com/eligrey/FileSaver.js/blob/master/FileSaver.js#L52 and their reasoning: https://code.google.com/p/chromium/issues/detail?id=375297#c7 |
@FesterCluck thanks will check it out |
Haven't seen RangeError in a while, but large downloads seem to stall part-way through (around when the tab hits 2GB ram?) |
You should check to see if you are hitting your machine's paging limit. |
For a 105MB file, the page uses 540MB of memory. By commenting out bits of the code at a time, I found this breakdown:
Checked off = fixed. We're creating lots of buffers that don't get released, it seems. Ideally, we only need to hang onto one copy of the buffer. |
Has memory leak fixes for https://github.com/feross/instant.io/issues/3
There are still some leaks, but it doesn't look like we're losing any full copies of the file anymore! Progress! There's only one copy of the file in memory. For a 360MB file:
cc @jhiesey |
We're now running into a bug in Chrome where blobs can't be larger than 500MB. https://code.google.com/p/chromium/issues/detail?id=375297 This looks like it's really close to being fixed though! Right now, this means you can't seed a file larger than 500MB from Chrome. I was able to seed a 1GB file in Firefox and stream to Chrome! |
_This issue was moved to webtorrent/webtorrent#319._ |
Large files fail with RangeError (in Chrome)
The text was updated successfully, but these errors were encountered: