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

https://github.com/dotnet/AspNetCore.Docs/issues/11801 #19497

Closed
wants to merge 1 commit into from

Conversation

sguitardude
Copy link
Contributor

Fixes #11801

I've tried to create a sample that synchronously adds log messages to an in-memory queue and has a background worker pull the messages out of the queue to do the asynchronous work of pushing data to SQL Server.

If the team agrees, then we can go back and update this article
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/logging/?view=aspnetcore-3.1#aspnet-core-and-ef-core-categories

@dnfadmin
Copy link

dnfadmin commented Aug 13, 2020

CLA assistant check
All CLA requirements met.

@guardrex
Copy link
Collaborator

Hello @sguitardude ... It's been challenging for the small team to keep samples up-to-date ... see #5495. In this case, third party providers are heavily recommended, and they use this (or similar) approaches for SQL Server. For example, I know Serilog accumulates and batch-writes to SQL Server. They'll let you know, but don't take it the wrong way if they have to decline. If they can't accept, it's probably just going to be for staffing/maintenance reasons in light of the popularity of third-party providers.

@sguitardude
Copy link
Contributor Author

sguitardude commented Aug 14, 2020 via email

@Rick-Anderson
Copy link
Contributor

@sguitardude can you put this on your GitHub? If so, I can tweet it.

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.

Guidance on how to log to a message queue for slow data stores
4 participants