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

[core] Stats: do not count discarded packets as dropped. #2932

Merged

Conversation

maxsharabayko
Copy link
Collaborator

Valid for a broadcast group member dropping existing packets from the RCV buffer because they were read from another member, not because they are missing. Such packets are now counted as 'discarded', and not included in the 'dropped' counter.

Fixes #2879.
Closes #2882.

@maxsharabayko maxsharabayko added Type: Bug Indicates an unexpected problem or unintended behavior [core] Area: Changes in SRT library core labels Apr 19, 2024
@maxsharabayko maxsharabayko added this to the v1.5.4 milestone Apr 19, 2024
Valid for a broadcast group member dropping existing packets from the RCV buffer because they were read from another member.
Co-authored-by: Sektor van Skijlen <ethouris@gmail.com>
@maxsharabayko maxsharabayko merged commit 4f925fb into Haivision:master Apr 22, 2024
12 checks passed
@maxsharabayko maxsharabayko deleted the hotfix/member-rcv-drop-cnt branch April 22, 2024 08:11
maxsharabayko added a commit to maxsharabayko/srt that referenced this pull request Apr 26, 2024
…).

Valid for a broadcast group member dropping existing packets from the RCV buffer because they were read from another member.

Co-authored-by: Sektor van Skijlen <ethouris@gmail.com>
maxsharabayko added a commit to maxsharabayko/srt that referenced this pull request Apr 26, 2024
…).

Valid for a broadcast group member dropping existing packets from the RCV buffer because they were read from another member.

Co-authored-by: Sektor van Skijlen <ethouris@gmail.com>
maxsharabayko added a commit that referenced this pull request Apr 29, 2024
Valid for a broadcast group member dropping existing packets from the RCV buffer because they were read from another member.

Co-authored-by: Sektor van Skijlen <ethouris@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[core] Area: Changes in SRT library core Type: Bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Statistics: pktRcvDrop for a broadcast group member
1 participant