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

StreamingEngine Parallel Segment Fetching Support #8

Closed
wants to merge 4 commits into from

Conversation

tyrelltle
Copy link
Owner

@tyrelltle tyrelltle commented Nov 24, 2022

This PR in in draft and will be reviewed by the shaka community soon.

Currently Shaka fetches the segments sequentially. The segment at future time will not be fetched until the segment at current time position is fetched / appended.

This PR adds the capaiblity of paralllel fetching the next N segments. This will reduce the chance of re-buffering especially when user plays at higher playback speed with limited network bandwidth.

image

this.stream_,
);
const endTime = startTime + duration;
for (const reference of this.segmentPrefetchMap_.keys()) {
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: use .values()

this.stream_ = stream;

/** @private {number} */
this.prefetchPosTime_ = 0;
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: change to clearer name

@github-actions
Copy link

Incremental code coverage: 52.73%

@tyrelltle
Copy link
Owner Author

Closing in favour of shaka-project#4784

@tyrelltle tyrelltle closed this Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant