Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

Define Mapping of Arbitrary Data to OTLP AnyValue #178

Conversation

tigrannajaryan
Copy link
Member

The mapping is necessary to correctly implement Logging Library SDKs such that
values mapped are unambiguous, consistent across languages and implementations.

@tigrannajaryan tigrannajaryan requested review from a team September 15, 2021 21:54
@tigrannajaryan tigrannajaryan force-pushed the object-mapping-to-otlp-anyvalue branch from 5e16dc7 to 971cb77 Compare September 15, 2021 21:55
@tigrannajaryan
Copy link
Member Author

@jkwatson @zenmoto PTAL.

text/logs/0178-mapping-to-otlp-anyvalue.md Outdated Show resolved Hide resolved
text/logs/0178-mapping-to-otlp-anyvalue.md Outdated Show resolved Hide resolved
@tigrannajaryan tigrannajaryan force-pushed the object-mapping-to-otlp-anyvalue branch from cde01da to ec3b2ba Compare September 17, 2021 01:51
@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-approvers Please review. This is needed by Logging SDK for now but there is no reason this cannot be extended to Span, Resource or SpanEvent attributes in the future if we decide so. It is best that we make it future-proof and ensure it is reasonable for all signals (even if we don't need it now).

@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-approvers Please review.

@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-logs-approvers please review.

text/logs/0178-mapping-to-otlp-anyvalue.md Outdated Show resolved Hide resolved
text/logs/0178-mapping-to-otlp-anyvalue.md Outdated Show resolved Hide resolved
text/logs/0178-mapping-to-otlp-anyvalue.md Outdated Show resolved Hide resolved
@tigrannajaryan tigrannajaryan force-pushed the object-mapping-to-otlp-anyvalue branch from ec3b2ba to d538bc2 Compare February 17, 2022 21:04
@tigrannajaryan
Copy link
Member Author

Lint failures will be fixed by #196 which needs to be merged before this PR.

@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-approvers please review.

The mapping is necessary to correctly implement Logging Library SDKs such that
values mapped are unambiguous, consistent across languages and implementations.
@tigrannajaryan tigrannajaryan force-pushed the object-mapping-to-otlp-anyvalue branch from 93eb9df to 82dec54 Compare February 18, 2022 16:46
@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-trace-approvers please review.

@tigrannajaryan
Copy link
Member Author

@zenmoto please review.

@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-metrics-approvers please review.

@tigrannajaryan
Copy link
Member Author

@open-telemetry/specs-approvers last chance to review/comment/object before I merge this and start moving to the spec. This is going to apply to all signals.

text/0178-mapping-to-otlp-anyvalue.md Outdated Show resolved Hide resolved
text/0178-mapping-to-otlp-anyvalue.md Show resolved Hide resolved
text/0178-mapping-to-otlp-anyvalue.md Show resolved Hide resolved
text/0178-mapping-to-otlp-anyvalue.md Show resolved Hide resolved
Copy link
Contributor

@MrAlias MrAlias left a comment

Choose a reason for hiding this comment

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

🚀

@tigrannajaryan tigrannajaryan merged commit aafcf0f into open-telemetry:main Feb 24, 2022
@tigrannajaryan tigrannajaryan deleted the object-mapping-to-otlp-anyvalue branch February 24, 2022 22:16
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Oct 23, 2024
The mapping is necessary to correctly implement Logging Library SDKs such that
values mapped are unambiguous, consistent across languages and implementations.
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Oct 23, 2024
The mapping is necessary to correctly implement Logging Library SDKs such that
values mapped are unambiguous, consistent across languages and implementations.
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Oct 30, 2024
The mapping is necessary to correctly implement Logging Library SDKs such that
values mapped are unambiguous, consistent across languages and implementations.
carlosalberto pushed a commit to open-telemetry/opentelemetry-specification that referenced this pull request Nov 8, 2024
…ps#178)

The mapping is necessary to correctly implement Logging Library SDKs such that
values mapped are unambiguous, consistent across languages and implementations.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants