Skip to content

Add option to disable log message queue behavior (performance issue)#96260

Closed
williammagno wants to merge 8 commits intodotnet:mainfrom
williammagno:master
Closed

Add option to disable log message queue behavior (performance issue)#96260
williammagno wants to merge 8 commits intodotnet:mainfrom
williammagno:master

Conversation

@williammagno
Copy link

When using K6 to do a performance test in an API with default console logger, the application can do 3.000 request in 60 seconds. When i remove the logger the application do 30.000 in 60 seconds. Removing the queue behavior from console logger i get the same 30.000 request result.

Add DisableMessageQueue option
Behavior to direct write message log without queue
Add DisableMessageQueue to processor constructor
@ghost ghost added area-Extensions-Logging community-contribution Indicates that the PR has been added by a community member labels Dec 21, 2023
@ghost
Copy link

ghost commented Dec 21, 2023

Tagging subscribers to this area: @dotnet/area-extensions-logging
See info in area-owners.md if you want to be subscribed.

Issue Details

When using K6 to do a performance test in an API with default console logger, the application can do 3.000 request in 60 seconds. When i remove the logger the application do 30.000 in 60 seconds. Removing the queue behavior from console logger i get the same 30.000 request result.

Author: williammagno
Assignees: -
Labels:

area-Extensions-Logging

Milestone: -

@huoyaoyuan
Copy link
Member

Thank you for your interest of contribution. Adding new API like methods require going through API review process.

@tarekgh
Copy link
Member

tarekgh commented Dec 26, 2023

Closing it as this need API proposal and go through the API review.

@tarekgh tarekgh closed this Dec 26, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Extensions-Logging community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants