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

Contribute otel-arrow-rust to OTEL #303

Merged
merged 4 commits into from
Mar 25, 2025

Conversation

v0y4g3r
Copy link
Contributor

@v0y4g3r v0y4g3r commented Mar 16, 2025

otel-arrow-rust is a Rust implementation of otel-arrow protocol. This implementation currently supports the univariate metrics signal. This contribution is considered a useful starting point for a complete Rust implementation of OTAP.

Part of #294
Part of open-telemetry/opentelemetry.io#6410

Originally authored at https://github.com/v0y4g3r/otel-arrow-rust

Copy link
Contributor

@jmacd jmacd left a comment

Choose a reason for hiding this comment

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

I reviewed this code in depth. I am a big fan, thank you @v0y4g3r.

As a relatively new Rust programmer, this gives me a substantial boost in productivity. I would be able to add support for the other signal types by following the example here for metrics.

Further, I am interested in benchmarking OTAP-to-OTLP (and later OTLP-to-OTAP) conversion functions; this will provide a valuable reference implementation. I believe as part of #294 our goal should be to optimize the OTAP-to-OTLP-bytes function directly--I'd like to experiment with encoding protobuf bytes directly from OTAP (i.e., w/o an intermediate prost::Message) and I would like to use this library as a starting point.

As I said for the Beaubourg contribution, #293 (review), there will be some degree of style differences in the repository in the short term, but nothing we can't fix as the project progresses.

@v0y4g3r v0y4g3r requested a review from drewrelmas March 18, 2025 06:35
@jmacd jmacd merged commit 296a456 into open-telemetry:main Mar 25, 2025
9 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.

3 participants