[bug][android] duplicated source requests sent in looped videos?

[lovegaoshi]

Bug

Platform

Which player are you experiencing the problem on:

• Android

Environment info

Library version: 6.0.0 beta2
Device: android 14 samsung S21, emulator

Steps To Reproduce

I don't see this behavior in 5.2.1, started to see this in v6; expo-av when ejected to bare workflow also has the same behavior if it's any relevant, but not when managed (??)

1. set a video to loop
2. monitor httpToolkit
3. the source (*.bilivideo.com) is periodically requested
   

...

Expected behaviour

the source is only requested once

Reproducible sample code

https://github.com/lovegaoshi/videotest/tree/rnv6

Video sample

If possible, include a link to the video that has the problem that can be streamed or downloaded from.

[lovegaoshi]

after some initial investigations:

this issue was describing the exact same behavior, however it was due to using exoplayer.setMediaItem instead of setMediaSource, when migrating away from the depreciated exoplayer.prepare(mediaSource). while rnv5.2.1 does use exoplayer.prepare, rnv6 also does use setMediaSource. using exoplayer.prepare in rnv6 doesnt magically fix the issue either.

then I'm considering this to be a problem with the media3 migration but the google media3 demo repo uses exoplayer.setMediaItems and I suspect this might be circling back to the problem described in the above issue; both media3 and exoplayer2 keep requesting the source as below:

I also tried to shove a simpleCache in RNV following this but while it did create the cache folder, nothing is actually stored within.

next i'll try to use setMediaSource in media3 and if this still occurs, I'll raise the issue to google; otherwise something was wrong in the RNV repo.

have no idea why expo has that weird behavior.

[lovegaoshi]

I got some clarifications from the media3 team and was able to make a caching system work; so not directly fixing this issue (bug?) but caching will effectively stop the repeated requests as well. I saw in the docs RNV only supports caching for iOS and from a quick repo search I dont see SimpleCache; is it still correct RNV doesnt have cache for android? I can implement that and migrate my app to RNV6 then.