-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
OpenTelemetry Protocol with Apache Arrow Receiver component #32015
Conversation
…tor-contrib into jmacd/arrow_receiver
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.
thanks for continuing to push this work forward @jmacd, just a couple of questions. Please resolve the conflicts and we can move this forward.
Will re-open this. |
…tor-contrib into jmacd/arrow_receiver
…tor-contrib into jmacd/arrow_receiver
…tor-contrib into jmacd/arrow_receiver
Needs a |
…tor-contrib into jmacd/arrow_receiver
Somehow |
@jmacd hmmm.... the same thing on my end, will take a look to see what's going on |
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@jmacd the problem is caused by CICD using go 1.21.9 when running make genotelcontribcol, which causes a diff in the
I've pushed the change, but if you run make genotelcontribcol locally, it will be set back to 1.22.2 😮💨 |
Long story short, if you can resolve the last conflict, we can get this merged and never think about it again. Sadly, this new |
…tor-contrib into jmacd/arrow_receiver
…-collector-contrib into jmacd/arrow_receiver
Description:
This is the same code as OTel-Arrow at 14c63d1eaac7c53585e6b9195d09f1f9703869ed, at which point lint fixes required for this repository were applied. Only import statements change here, to match the host repository.
Link to tracking Issue: #26491
Testing: Test coverage is approximately 90%.
Documentation: I double-checked and the existing README, no changes required.