Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Exoplayer offline download consumes more data than file size #5927

Closed
ruthwikkk opened this issue May 22, 2019 · 3 comments
Closed

Exoplayer offline download consumes more data than file size #5927

ruthwikkk opened this issue May 22, 2019 · 3 comments
Assignees
@erdemguven
Labels
bug

Comments

@ruthwikkk
Copy link

ruthwikkk commented May 22, 2019 •
edited
Loading

[REQUIRED] Searched documentation and issues
I am using ExoPlayer version 2.9.6 to stream video content in one of my project. Recently I added download video feature to my app using ExoPlayer's in built download manager. The problem with download is that it consumes more data while downloading. For a 18 MB video (the size is determined by checking the size of ExoPlayer's download folder after download) , it takes about 140 MB (this is variable in each try) of network data. I am using DASH to stream videos. I checked the same with Demo App provided and the results are almost same.

[REQUIRED] Question
Any one know why this is happening.? Is there any improvements needed in my code ?
@erdemguven erdemguven self-assigned this May 22, 2019
@google google deleted a comment from google-oss-bot May 22, 2019
@erdemguven
Copy link
Contributor

How do you measure the used network data?
Could you share the measurement you did using the ExoPlayer demo app?

@ruthwikkk
Copy link
Author

How do you measure the used network data?
Could you share the measurement you did using the ExoPlayer demo app?

I noticed this issue only after some of our users started reporting. I measured the network data using GlassWire network monitor app. I have downloaded a 37 MB video using ExoPlayer Demo app and here I am attaching the screenshot from data monitor app

Screenshot_20190522-203711

@erdemguven
Copy link
Contributor

I can confirm the issue now. Soon a fix will be committed. Thanks for reporting.

tonihei pushed a commit that referenced this issue May 30, 2019
@erdemguven @tonihei
Fix CacheUtil.cache() use too much data
42ba6ab 
cache() opens all connections with unset length to avoid position errors.
This makes more data then needed to be downloading by the underlying
network stack.

This fix makes makes it open connections for only required length.

Issue:#5927
PiperOrigin-RevId: 250546175
@ojw28 ojw28 closed this as completed May 31, 2019
ojw28 pushed a commit that referenced this issue Jun 3, 2019
@erdemguven @ojw28
Fix CacheUtil.cache() use too much data
92e2581 
cache() opens all connections with unset length to avoid position errors.
This makes more data then needed to be downloading by the underlying
network stack.

This fix makes makes it open connections for only required length.

Issue:#5927
PiperOrigin-RevId: 250546175
@google google locked and limited conversation to collaborators Oct 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@erdemguven erdemguven

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ojw28 @erdemguven @ruthwikkk