Skip to content

Switch to Rust native async #2517

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.

Sign up for GitHub

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

Jump to bottom
Closed
cijothomas opened this issue Jan 15, 2025 · 3 comments
Closed

Switch to Rust native async #2517

cijothomas opened this issue Jan 15, 2025 · 3 comments
Assignees
@lalitb
Milestone
0.29

Comments

@cijothomas
Copy link
Member

#2374 did this for Logs. Need to replicate this in other places.
@cijothomas cijothomas added this to the Metrics SDK Stable milestone Jan 15, 2025
@cijothomas cijothomas added the help wanted Good for taking. Extra help will be provided by maintainers/approvers label Jan 15, 2025
@lalitb lalitb self-assigned this Jan 17, 2025
@lalitb lalitb removed the help wanted Good for taking. Extra help will be provided by maintainers/approvers label Jan 17, 2025
This was referenced Feb 13, 2025
Merged
Merged
@cijothomas
Copy link
Member Author

https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-sdk/src/trace/export.rs#L31 Export method in Tracing signal can be modified to avoid the Box ? @lalitb was exploring this..

@lalitb
Copy link
Member

lalitb commented Feb 15, 2025

Export method in Tracing signal can be modified to avoid the Box ? @lalitb was exploring this..

Yes, I have the changes locally. Unless some-one wants to do it earlier, I can raise a PR next week.

@cijothomas cijothomas modified the milestones: Tracing SDK Stable, 0.29 Feb 18, 2025
@cijothomas
Copy link
Member Author

Closed with #2685 and #2672

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lalitb lalitb

Labels
None yet
Projects
None yet
Milestone
0.29
Development

No branches or pull requests
2 participants
@lalitb @cijothomas