-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Enhance Kafka exporter to respect max message size #6439
Comments
@yurishkuro I would like to contribute to this issue can you please assign me this? |
consider it assigned |
Hi @yurishkuro , I am just informing you that I'm working on this issue, I came down with a cold last week, I am sorry for the delay. |
@yurishkuro This issue seems to be in limbo. So I would also like to work on this. @Riteek712 Can we contribute together on this issue? |
@ADI-ROXX sure, can you mention your email for correspondence? |
@yurishkuro Can you provide some commands to duplicate the problem on our own system? |
hey folks, I'm working on this as it is needed for us, the splitting logic for traces is not straight forward for spans, had to do binary search over spans. Here's a PR open-telemetry/opentelemetry-collector-contrib#37176, I have added unit tests. |
Okay @shivanshuraj1333 👍 |
This is a copy of the upstream issue open-telemetry/opentelemetry-collector-contrib#36982 that we encountered, creating a ticket here to track and for better discoverability.
Problem is described here: #6437 (comment)
The text was updated successfully, but these errors were encountered: