Skip to content

Add support for Boolean8 to NetTrace V6.#2318

Merged
brianrob merged 1 commit intomicrosoft:mainfrom
noahfalk:bool8
Oct 16, 2025
Merged

Add support for Boolean8 to NetTrace V6.#2318
brianrob merged 1 commit intomicrosoft:mainfrom
noahfalk:bool8

Conversation

@noahfalk
Copy link
Member

EventSource writes booleans differently in manifest mode vs self-describing mode. Manifest mode uses a 4 byte boolean event field which NetTrace has always supported but Self-describing wrote a 1 byte boolean which NetTrace had no metadata encoding for. Adding support for Boolean8 to NetTrace gives us a path forward to properly describe booleans in metadata regardless of mode.

EventSource writes booleans differently in manifest mode vs self-describing mode. Manifest mode uses a 4 byte boolean event field which NetTrace has always supported but Self-describing wrote a 1 byte boolean which NetTrace had no metadata encoding for. Adding support for Boolean8 to NetTrace gives us a path forward to properly describe booleans in metadata regardless of mode.
Copy link
Member

@brianrob brianrob left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @noahfalk!

@brianrob brianrob merged commit 79f04f4 into microsoft:main Oct 16, 2025
5 checks passed
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

Comments