这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@qingling128
Copy link

No description provided.

@google-cla google-cla bot added the cla: yes label Jan 15, 2021
@qingling128 qingling128 changed the title Update the api request size limit to reflect the latest number. Update the api request size limit from 5MB to 10MB. Jan 15, 2021
# 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.

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.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: biggers.

@igorpeshansky igorpeshansky changed the title Update the api request size limit from 5MB to 10MB. Clarify that the API request size limit is 10MB, not 5MB. Jan 15, 2021
Copy link
Author

@qingling128 qingling128 left a 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.

Copy link

@igorpeshansky igorpeshansky left a 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.

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?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

Copy link
Author

@qingling128 qingling128 left a 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.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

Copy link

@igorpeshansky igorpeshansky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

@qingling128 qingling128 merged commit bb9d476 into master Jul 14, 2021
@qingling128 qingling128 deleted the lingshi-size branch July 14, 2021 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants