Skip to content

Conversation

@minherz
Copy link
Contributor

@minherz minherz commented Sep 12, 2021

provide an explanation about synchronous and asynchronous modes for writing log entries and how the mode can be configured.

Fixes #665

@minherz minherz requested review from a team as code owners September 12, 2021 11:01
@product-auto-label product-auto-label bot added the api: logging Issues related to the googleapis/java-logging API. label Sep 12, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 12, 2021
@minherz
Copy link
Contributor Author

minherz commented Sep 12, 2021

Unfortunately, the library does not provide a way to configure batching settings. It does not make sense to change the code in order to expose batching settings now. This change will be incorporated into the planed refactoring of the existing batching implementation.

Copy link
Contributor

@daniel-sanche daniel-sanche left a comment

Choose a reason for hiding this comment

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

LGTM, but Simon should also probably take a look since he's more familiar with the java library than I am

@minherz minherz merged commit e749acc into master Sep 14, 2021
@minherz minherz deleted the minherz/fix-665 branch September 14, 2021 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging Issues related to the googleapis/java-logging API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

doc: documentation does not describe different modes for writing log entries

3 participants