-
Notifications
You must be signed in to change notification settings - Fork 305
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
[Geneva] Optimize metrics exporter serializer #2460
base: main
Are you sure you want to change the base?
Conversation
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Should I update the changelog also or what else is missing from the PR? |
@TimothyMothra, @xiang17, FYI |
I will review the PR in couple of days. |
There is a test failure, could you please check? |
Higher-level changes include serializing resource level tags only once for
OtlpProtobufMetricExporter
.Most of the changes are just low-level optimizations for efficient buffer access, single-pass string encoding, etc.