-
Notifications
You must be signed in to change notification settings - Fork 50
Clarify that the API request size limit is 10MB, not 5MB. #313
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
Conversation
7f481ae to
089ef53
Compare
templates/etc/td-agent/td-agent.conf
Outdated
| # Set the chunk limit conservatively to avoid exceeding the recommended | ||
| # chunk size of 5MB per write request. | ||
| # chunk size of 10MB per write request. The API request size can be a few | ||
| # times biggers than the raw log size. |
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.
Typo: biggers.
| # Set the chunk limit conservatively to avoid exceeding the recommended | ||
| # chunk size of 5MB per write request. | ||
| # chunk size of 10MB per write request. The API request size can be a few | ||
| # times biggers than the raw log size. |
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.
Typo: biggers.
qingling128
left a comment
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.
Nice catch! Fixed the typo.
igorpeshansky
left a comment
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.
One more place for this change.
| buffer_type file | ||
| buffer_path /var/log/google-fluentd/buffers | ||
| # Set the chunk limit conservatively to avoid exceeding the recommended | ||
| # chunk size of 5MB per write request. |
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.
Do we need the same change in windows-installer/fluent-template.conf?
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.
Updated.
0512b52 to
1bd2c02
Compare
qingling128
left a comment
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.
PTAL
| buffer_type file | ||
| buffer_path /var/log/google-fluentd/buffers | ||
| # Set the chunk limit conservatively to avoid exceeding the recommended | ||
| # chunk size of 5MB per write request. |
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.
Updated.
igorpeshansky
left a comment
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
No description provided.