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

Logging in the Package #362

Open
ardalis opened this issue Aug 16, 2023 · 4 comments
Open

Logging in the Package #362

ardalis opened this issue Aug 16, 2023 · 4 comments

Comments

@ardalis
Copy link
Owner

ardalis commented Aug 16, 2023

Based on these docs:
https://learn.microsoft.com/en-us/dotnet/core/extensions/logging-library-authors

Is there any logging that would be worthwhile for users of the Specification package (or its Repository base types)?

@fiseni
Copy link
Collaborator

fiseni commented Aug 16, 2023

Not really. The EF will emit logs anyway. Since we're using consumers' DbContext, the logging is included based on their respective configuration.
Other than that, I can't think of anything worthwhile.

@ardalis
Copy link
Owner Author

ardalis commented Aug 16, 2023

I sometimes create logging decorators for repos that include timers so I can determine which queries are slow - might that be useful? EF logging doesn't provide this in a nice manner.

@fiseni
Copy link
Collaborator

fiseni commented Aug 16, 2023

What would the logging content look like in that case? Also, will you depend on DateTime?

@fiseni
Copy link
Collaborator

fiseni commented Nov 21, 2023

Now that the OTEL is becoming a standard, perhaps we should add distributed tracing instrumentation using Activity. That makes more sense than logging. Also, we may publish metrics, that might be useful for perf analysis. We need to analyze and invest in this.

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

No branches or pull requests

2 participants