Skip to content

Fix logger lost messages V2 #2789

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

Open
wants to merge 1 commit into
base: yarp-3.12
Choose a base branch
from

Conversation

randaz81
Copy link
Member

LogForwarder: now using a separated thread (class ThreadedPort) to prevent the loss of log messages during stress condition

@randaz81 randaz81 self-assigned this Dec 28, 2021
@randaz81 randaz81 requested a review from drdanz as a code owner December 28, 2021 10:00
@randaz81 randaz81 marked this pull request as draft December 28, 2021 10:01
@randaz81 randaz81 linked an issue Dec 28, 2021 that may be closed by this pull request
@randaz81 randaz81 force-pushed the master branch 3 times, most recently from 71900d2 to ead1caf Compare November 30, 2023 12:46
@randaz81 randaz81 force-pushed the fix_logForwareder_lost_msgs_v2 branch from 21f79d9 to 3389657 Compare June 27, 2025 10:26
@randaz81 randaz81 marked this pull request as ready for review July 29, 2025 21:17
@randaz81 randaz81 force-pushed the fix_logForwareder_lost_msgs_v2 branch from 3389657 to e7f70b6 Compare July 29, 2025 21:17
@randaz81 randaz81 changed the base branch from master to yarp-3.12 July 31, 2025 14:56
@randaz81
Copy link
Member Author

/rebase

@randaz81 randaz81 force-pushed the fix_logForwareder_lost_msgs_v2 branch from e7f70b6 to 8363f5d Compare August 1, 2025 12:18
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
68.9% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@randaz81
Copy link
Member Author

To be rebased and tested after merging PR #3273

…prevent the loss of log messages during stress condition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Yarplogger may not report all entries
1 participant