-
Notifications
You must be signed in to change notification settings - Fork 183
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
Adjust queueing and batching configurarion for OTC according to testi… #2474
Adjust queueing and batching configurarion for OTC according to testi… #2474
Conversation
@@ -3977,7 +3977,7 @@ metadata: | |||
## ref: https://github.com/SumoLogic/opentelemetry-collector/blob/main/processor/batchprocessor | |||
batch: | |||
## Number of spans after which a batch will be sent regardless of time | |||
send_batch_size: 16_384 | |||
send_batch_size: 1_024 | |||
## Time duration after which a batch will be sent regardless of size | |||
timeout: 30s |
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.
Should we update timeout as well?
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.
Yeah, a lower value like 1s
is better imo.
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.
I will update that
ef1d6f8
to
9f8b086
Compare
9f8b086
to
0c61160
Compare
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, do you plan on doing logs in a separate PR?
From what I see, originally it was done only for metrics, so I updated only metrics. I can prepare separate PR for logs |
0c61160
to
b1812cc
Compare
…ng results
Description
Fill in your description here.
Checklist
Testing performed