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

fix: log prefetch miss for missing segments #6012

Merged
merged 1 commit into from Jan 8, 2024

Conversation

gkatsev
Copy link
Contributor

@gkatsev gkatsev commented Dec 12, 2023

A recent change made it always log that segments were re-used, but in reality there were times when they were missed. Uses the language from the original PR

A recent change made it always log that segments were re-used, but in reality there were times when they were missed. Uses the language from the original PR
@shaka-bot
Copy link
Collaborator

Incremental code coverage: 50.00%

@avelad avelad added type: bug Something isn't working correctly priority: P2 Smaller impact or easy workaround labels Dec 23, 2023
@avelad avelad added this to the v5.0 milestone Dec 23, 2023
@avelad avelad merged commit a70136d into shaka-project:main Jan 8, 2024
17 checks passed
avelad pushed a commit that referenced this pull request Jan 8, 2024
A recent change made it always log that segments were re-used, but in
reality there were times when they were missed. Uses the language from
the original PR
avelad pushed a commit that referenced this pull request Jan 8, 2024
A recent change made it always log that segments were re-used, but in
reality there were times when they were missed. Uses the language from
the original PR
@gkatsev gkatsev deleted the patch-1 branch January 8, 2024 14:14
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Mar 8, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Mar 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants