Skip to content
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

[content/en/opentelemetry] Mention different signals request different batch processor configs #16600

Merged
merged 3 commits into from
Jan 25, 2023

Conversation

songy23
Copy link
Member

@songy23 songy23 commented Jan 24, 2023

What does this PR do?

Remind the readers that they may need different batch sizes for batch processor for different signal types (traces/metrics/logs) when using datadog exporter.

Motivation

The previous comment and config in the batch processor only refers to the payload size limit of trace intake. When customers use the same config for metrics or logs pipeline, they may get 413 errors because metrics and logs have a smaller payload size limit than that in trace. This led to customer confusions and issues being filed in https://github.com/open-telemetry/opentelemetry-collector-contrib.

Additional Notes

Related customer issues:

Also note that the payload size limits in metrics and logs intake are in the public document, but traces intake doesn't have a public reference.


Reviewer checklist

  • Review the changed files.
  • Review the URLs listed in the Preview section.
  • Check images for PII
  • Review any mentions of "Contact Datadog support" for internal support documentation.

@songy23 songy23 requested a review from a team as a code owner January 24, 2023 19:26
@github-actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

songy23 and others added 2 commits January 24, 2023 16:59
Co-authored-by: Ursula Chen <58821586+urseberry@users.noreply.github.com>
@urseberry urseberry merged commit bea2fbd into master Jan 25, 2023
@urseberry urseberry deleted the songy23/otel-collector-batch-config branch January 25, 2023 17:54
@songy23
Copy link
Member Author

songy23 commented Jan 25, 2023

Thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants