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

Structured Logs definition #267

Closed
wants to merge 1 commit into from

Conversation

MSNev
Copy link

@MSNev MSNev commented Oct 1, 2024

With the definition of Events there has been additional contentious and confusion around the difference between what IS and Event vs a Structured Log when they are both represented and transported as a type of OpenTelemetry Log.

This also avoids overloading of the "Events" definition to include Structured Logs, as they have different intents and purposes.

And this avoids the "Hack" approach like that used for Zero length Spans to simulate Events, before Events had been defined. With the introduction of Events and support in the OpenTelemetry Semantic Convention tooling it is an extremely trivial task to define a new type definition for Structured Logs whch can be used to define the structure of the log within Semantic Conventions yaml and avoiding the multiple needs of overloading the term "Event", overloading the usage of the event.name and to support the Semantic differences between Events and Structured Logs.

Like supporting "Unnamed" Structured Logs, which would not be possible with the current Event definition as events MUST have a name field and all events with the same name are comparable.

@MSNev
Copy link
Author

MSNev commented Oct 1, 2024

We have decided in the Logs Sig to not support the production of specific Structured Logs at this point.

@MSNev MSNev closed this Oct 1, 2024
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.

1 participant