After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 789457 - adaptivedemux: add replaced flag to not error out on bitrate change
adaptivedemux: add replaced flag to not error out on bitrate change
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other All
: Normal normal
: 1.13.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 789084 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2017-10-25 06:55 UTC by Thiago Sousa Santos
Modified: 2017-10-27 10:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
adaptivedemux: add replaced flag to not error out on bitrate change (2.87 KB, patch)
2017-10-25 06:55 UTC, Thiago Sousa Santos
committed Details | Review

Description Thiago Sousa Santos 2017-10-25 06:55:26 UTC
With HLS it is common to have a live stream abort when it is trying to download ahead of the last fragment and it tries to switch bitrate at the same time.

This fixes the problem by having an extra flag to signal the stream is being replaced due to a bitrate change rather than an error. So we can differentiate and avoid erroring out mistakenly
Comment 1 Thiago Sousa Santos 2017-10-25 06:55:29 UTC
Created attachment 362236 [details] [review]
adaptivedemux: add replaced flag to not error out on bitrate change

When switching bitrates we set the old streams as cancelled, but it
could also be confused with a cancel due to other reasons (as an error)
and it would lead the element to stop the pipeline mistankely. This
would happen when the stream being replaced was waiting for a manifest
update on live. Ss make it sure that we are stopping for switching
bitrates to avoid erroring out.
Comment 2 Florian Zwoch 2017-10-26 08:09:47 UTC
*** Bug 789084 has been marked as a duplicate of this bug. ***
Comment 3 Thiago Sousa Santos 2017-10-26 08:12:25 UTC
commit c2d34d1e44fbe6c4a0e7792b8d22e1059a4b1db7
Author: Thiago Santos <thiagossantos@gmail.com>
Date:   Sat Oct 14 13:22:18 2017 -0700

    adaptivedemux: add replaced flag to not error out on bitrate change
    
    When switching bitrates we set the old streams as cancelled, but it
    could also be confused with a cancel due to other reasons (as an error)
    and it would lead the element to stop the pipeline mistankely. This
    would happen when the stream being replaced was waiting for a manifest
    update on live. Ss make it sure that we are stopping for switching
    bitrates to avoid erroring out.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=789457
Comment 4 Florian Zwoch 2017-10-27 10:35:50 UTC
I have applied this patch to my local 1.12.3 and it works with offsets. I have run it for a couple of hours without any "Could not download fragments" errors. I got these once in while quite 'reliably' before over wireless.