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

Avoid allocating CompositeLogger and related components when disabled #208

Open
stevejgordon opened this issue Feb 10, 2025 · 0 comments
Open
Milestone

Comments

@stevejgordon
Copy link
Contributor

When logging options do not require logging in any form, we should consider using the shared NullLogger instance to avoid allocating the logging infrastructure that is unused. This will require refactoring some internal APIs to prefer ILogger and may not be entirely possible.

This issue should serve as a reminder for an investigation into the impact and whether this is possible.

The allocation overhead is low so this is low priority.

@stevejgordon stevejgordon added this to the Future milestone Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant