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.

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

[Proposal] [WIP] Improve multi-URL priorization #1165

Merged
merged 6 commits into from
Oct 5, 2022

Conversation

peaBerberian
Copy link
Collaborator

Our work toward implementing ContentSteering (#1142) also brought as a side-effect a better CDN priorization algorithm, when multiple ones are available for a segment:
A request returning with an error will result with the corresponding CDN being temporarily "downgraded" for 60 seconds. The corresponding CDN would then be avoided when possible for this amount of time, regardless of the resource in question.

This seems to be very useful for all cases where multiple CDNs are available and not just when relying on Content Steering - which still seems to be in a draft phase. Consequently this PR removes all ContentSteering-related bits from that previous PR (it may be merged in the future) to only let the code:

  • identifying a particular "CDN" when multiple ones are available, by relying in DASH on the corresponding absolute BaseURL
  • prioritizing CDNs between one another: first by initially relying on the first one announced in the Manifest, considering the next URL only once an error has been encountered on the previous one - until its "downgrade" delay is expired (previously we tried each CDN in order per-segment, meaning that the problematic CDN would be retried on each segment request).

@peaBerberian peaBerberian added Priority: 3 (Low) This issue or PR has a low priority. skip-performance-checks Performance tests are not run on this PR labels Sep 28, 2022
@peaBerberian peaBerberian added this to the 3.29.0 milestone Sep 28, 2022
@peaBerberian peaBerberian force-pushed the next branch 2 times, most recently from 2a76f8a to 79d5c62 Compare October 3, 2022 15:48
@peaBerberian peaBerberian force-pushed the misc/better-multi-url-priorization branch 2 times, most recently from 0a3383a to 6a42b71 Compare October 4, 2022 09:47
@peaBerberian peaBerberian added skip-performance-checks Performance tests are not run on this PR and removed skip-performance-checks Performance tests are not run on this PR labels Oct 4, 2022
@peaBerberian peaBerberian force-pushed the misc/better-multi-url-priorization branch from 6a42b71 to f9ca327 Compare October 5, 2022 14:26
@peaBerberian peaBerberian merged commit ce83e41 into next Oct 5, 2022
peaBerberian added a commit that referenced this pull request Oct 13, 2022
…ation

[Proposal] [WIP] Improve multi-URL priorization
peaBerberian added a commit that referenced this pull request Oct 28, 2022
…ation

[Proposal] [WIP] Improve multi-URL priorization
peaBerberian added a commit that referenced this pull request Oct 28, 2022
…ation

[Proposal] [WIP] Improve multi-URL priorization
@peaBerberian peaBerberian mentioned this pull request Nov 15, 2022
@peaBerberian peaBerberian deleted the misc/better-multi-url-priorization branch July 6, 2023 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 3 (Low) This issue or PR has a low priority. skip-performance-checks Performance tests are not run on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant