Skip to content

Updates to XEvents sessions #2578

Merged
allancascante merged 3 commits intomainfrom
dev/allancascante/xevent_session_updates
Feb 27, 2026
Merged

Updates to XEvents sessions #2578
allancascante merged 3 commits intomainfrom
dev/allancascante/xevent_session_updates

Conversation

@allancascante
Copy link
Contributor

@allancascante allancascante commented Feb 2, 2026

Description

Updates to comments to use Event sessions name in the profiler code. Removed the reconnect logic, on failure just notify of the problems.

Code Changes Checklist

  • New or updated unit tests added
  • All existing tests pass (dotnet test)
  • Code follows contributing guidelines
  • Logging/telemetry updated if relevant
  • No protocol or behavioral regressions

Reviewers: Please read our reviewer guidelines

kburtram
kburtram previously approved these changes Feb 26, 2026
Copy link
Member

@kburtram kburtram left a comment

Choose a reason for hiding this comment

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

This makes sense to me to remove the connection retry logic. The CI build is failing and should be investigated prior to merge to confirm it's unrelated.

@allancascante allancascante merged commit 5dc3b92 into main Feb 27, 2026
5 checks passed
@allancascante allancascante deleted the dev/allancascante/xevent_session_updates branch February 27, 2026 19:54
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.

2 participants