Skip to content

Fix: fragmentSize is below the minimum#6238

Merged
TobiGr merged 2 commits intodevfrom
fixfragmentsize
May 4, 2021
Merged

Fix: fragmentSize is below the minimum#6238
TobiGr merged 2 commits intodevfrom
fixfragmentsize

Conversation

@Redirion
Copy link
Member

@Redirion Redirion commented May 4, 2021

noticed logs " CacheDataSink: fragmentSize is below the minimum recommended value of 2097152. This may cause poor cache performance.

this fixes the issue by using ExoPlayers MIN_RECOMMENDED_FRAGMENT_SIZE. Unfortunately that field is private:
https://github.com/google/ExoPlayer/blob/aeb306a164911aa1491b46c2db4da0d329c83c65/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheDataSink.java#L123

What is it?

  • Codebase improvement (dev facing)

Fixes the following issue(s)

Discovered in #6207

APK testing

On the website the APK can be found by going to the "Checks" tab below the title and then on "artifacts" on the right.

Due diligence

noticed logs " CacheDataSink: fragmentSize is below the minimum recommended value of 2097152. This may cause poor cache performance.

this fixes the issue by using ExoPlayers MIN_RECOMMENDED_FRAGMENT_SIZE. Unfortunately that field is private:
https://github.com/google/ExoPlayer/blob/aeb306a164911aa1491b46c2db4da0d329c83c65/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheDataSink.java#L123
@TobiGr TobiGr added the player Issues related to any player (main, popup and background) label May 4, 2021
@TobiGr TobiGr merged commit c85b97a into dev May 4, 2021
@TobiGr TobiGr deleted the fixfragmentsize branch May 4, 2021 17:08
This was referenced May 23, 2021
spvkgn pushed a commit to spvkgn/NewPipe that referenced this pull request Jun 5, 2021
noticed logs " CacheDataSink: fragmentSize is below the minimum recommended value of 2097152. This may cause poor cache performance.

this fixes the issue by using ExoPlayers MIN_RECOMMENDED_FRAGMENT_SIZE. Unfortunately that field is private:
https://github.com/google/ExoPlayer/blob/aeb306a164911aa1491b46c2db4da0d329c83c65/library/core/src/main/java/com/google/android/exoplayer2/upstream/cache/CacheDataSink.java#L123
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

player Issues related to any player (main, popup and background)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants