Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

[monitoring] refactor monitoring#763

Merged
justonedev1 merged 1 commit intomasterAliceO2Group/Control:masterfrom
refactor_monitoringAliceO2Group/Control:refactor_monitoringCopy head branch name to clipboard
Nov 18, 2025
Merged

[monitoring] refactor monitoring#763
justonedev1 merged 1 commit intomasterAliceO2Group/Control:masterfrom
refactor_monitoringAliceO2Group/Control:refactor_monitoringCopy head branch name to clipboard

Conversation

@justonedev1
Copy link
Collaborator

Lately there was a case when internal monitoring wasn't running. So I changed life of internals for the whole time of application, so we don't have to synchronize Start and Stop and Send and just consume. Thanks to this I could drop atomic load in Sending.
I also added WaitUntilRunning mostly for informative purposes it is not 100% necessary for correct function of the library, but it is really helpful for testing.

@justonedev1 justonedev1 requested a review from knopers8 as a code owner October 31, 2025 09:04
@justonedev1 justonedev1 merged commit 9258177 into master Nov 18, 2025
4 checks passed
@justonedev1 justonedev1 deleted the refactor_monitoring branch November 18, 2025 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.