-
Notifications
You must be signed in to change notification settings - Fork 888
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
Clarify that lowerCamelCase field names MUST be used for OTLP/JSON #2829
Clarify that lowerCamelCase field names MUST be used for OTLP/JSON #2829
Conversation
871c911
to
f5a713c
Compare
I discussed this with @yurishkuro. He has a very good argument that we control most of OTLP producers (the SDKs and Collector), while we don't control most of OTLP consumers (e.g. the backends).
I will update this PR to reflect this thinking and will reset the existing approvals. |
f5a713c
to
72df625
Compare
2218e6e
to
3be7daa
Compare
Let's give this another 24 hours and then merge if there are no objections. |
Resolves open-telemetry#2795 This is a breaking change for OTLP/JSON and is allowed because OTLP/JSON is not yet Stable.
ddaeb2b
to
2ebe1fd
Compare
Has enough approvals. No objections for several days. Merging. |
…pen-telemetry#2829) Resolves open-telemetry#2795 This is a breaking change for OTLP/JSON and is allowed because OTLP/JSON is not yet Stable.
Resolves #2795
This is a breaking change for OTLP/JSON and is allowed because OTLP/JSON is not yet Stable.