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

Revamp the Asynchronous logging page #2577

Merged
merged 24 commits into from
May 21, 2024
Merged

Revamp the Asynchronous logging page #2577

merged 24 commits into from
May 21, 2024

Conversation

ppkarwasz
Copy link
Contributor

We refresh the asynchronous logging page by:

  • marking more clearly that asynchronous logging is not the solution to all performance problems. Depending on the case it could give all disadvantage of async processing without any measurable advantages.
  • removes the system properties and points to the appropriate "Configuration properties" section.
  • refreshes the configuration examples by adding an example for each of the configuration formats.

Closes #2534.

@ppkarwasz ppkarwasz marked this pull request as ready for review May 15, 2024 11:45
@ppkarwasz ppkarwasz requested review from grobmeier and vy May 15, 2024 11:45
We refresh the asynchronous logging page by:

* marking more clearly that _asynchronous logging_ is not the solution
  to all performance problems. Depending on the case it could give all
  disadvantage of async processing without any measurable advantages.
* removes the system properties and points to the appropriate
  "Configuration properties" section.
* refreshes the configuration examples by adding an example for each of
  the configuration formats.

Closes #2534.
@ppkarwasz ppkarwasz changed the base branch from 2.x to doc/perf May 20, 2024 10:58
Base automatically changed from doc/perf to 2.x May 21, 2024 07:54
@ppkarwasz ppkarwasz requested a review from vy May 21, 2024 10:25
@ppkarwasz ppkarwasz merged commit f4cf44e into 2.x May 21, 2024
9 checks passed
@ppkarwasz ppkarwasz deleted the doc/2.x/async branch May 21, 2024 11:12
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.

Revamp the Asynchronous logging page
2 participants