-
Notifications
You must be signed in to change notification settings - Fork 164
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
Columnar encoding for the OpenTelemetry protocol #171
Conversation
Improve conclusion
Co-authored-by: Reiley Yang <reyang@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with some minor wording suggestions.
Co-authored-by: Reiley Yang <reyang@microsoft.com>
Co-authored-by: Reiley Yang <reyang@microsoft.com>
Co-authored-by: Reiley Yang <reyang@microsoft.com>
Co-authored-by: Reiley Yang <reyang@microsoft.com>
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@jsuereth @zeroshade @mbrobbel @itaykat could you take another look and see if there is anything blocking? If not, I suggest that we give the PR few more days and merge it by end of this week (6/30/2023). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
The GC voted to accept the donation (open-telemetry/community#1332) on July 6 2023 |
Did we ever get results back on the Phase 2 multivariate time series tests?
|
@infogulch - The current implementation (Phase 1) includes optimizations for multivariate metrics, which are presented as a set of univariate metrics. We've observed a compression ratio enhancement ranging from 3x to 7x with this setup. Phase 2, which is yet to start, will deepen this support for multivariate metrics, integrating them not just at the protocol layer but also within the client SDK API. I aim to release a beta version of this extended client SDK by the end of the year. |
I see, thank you for clarifying. I didn't understand that the phase 2 tests are still in process. Personally I've been watching otel for a while but am not currently using it; my comment came from a fleeting curiosity about the intersection of telemetry and column-oriented storage/processing formats from the recent hn thread about otel, where someone kindly forwarded me here. From this report it seems they work together quite nicely! (I love when incidental ideas are confirmed by someone that put in the work to test it 6 months ago!) Thank you for your attention, sorry for the noise, and good luck integrating columnar layouts into otel! |
Ongoing experiments, research, and draft specification for a multi-variate columnar encoding of OTLP.
At the request of @jmacd