Skip to content

VPLAY-12736: Add debug logs when latency is printed as negative#1038

Merged
pstroffolino merged 3 commits intodev_sprint_25_2from
feature/VPLAY-12736
Feb 18, 2026
Merged

VPLAY-12736: Add debug logs when latency is printed as negative#1038
pstroffolino merged 3 commits intodev_sprint_25_2from
feature/VPLAY-12736

Conversation

@srikanthreddybijjam-comcast
Copy link
Copy Markdown
Contributor

@srikanthreddybijjam-comcast srikanthreddybijjam-comcast commented Feb 17, 2026

Reason for change: Added logs when negative latency occurs
Risks: Low
Test Procedure: Refer jira ticket VPLAY-12736
Priority: P1

Copilot AI review requested due to automatic review settings February 17, 2026 05:36
@srikanthreddybijjam-comcast srikanthreddybijjam-comcast requested a review from a team as a code owner February 17, 2026 05:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds diagnostic logging to detect negative latency calculations in the AAMP player's progress monitoring system. The changes specifically target a production issue (VPLAY-12736) where negative latency values are being reported after trickplay operations to the live edge.

Changes:

  • Added conditional logging for negative latency values in both DASH and HLS live stream scenarios
  • Logs include detailed diagnostic information (latency value and contributing variables) to help identify the root cause of latency calculation overflow/underflow

@srikanthreddybijjam-comcast srikanthreddybijjam-comcast changed the title VPLAY-12736: After trickplay to live edge and playback is resumed, Latency Printed as negative. VPLAY-12736: Add debug logs when latency is printed as negative Feb 17, 2026
…tency value is printed negative

Reason for change: Added Logs for Negative Latency
Test Procedure: Refer jira ticket VPLAY-12270
Priority: P1

Signed-off-by: srikanthreddybijjam-comcast <srikanthreddybijjam.2000@gmail.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings February 18, 2026 13:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread priv_aamp.cpp
Comment thread priv_aamp.cpp
@pstroffolino pstroffolino merged commit 65c8dd5 into dev_sprint_25_2 Feb 18, 2026
9 of 11 checks passed
@pstroffolino pstroffolino deleted the feature/VPLAY-12736 branch February 18, 2026 15:53
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.

3 participants