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

Allow injecting logger or logger factory #190

Closed
xerial opened this issue Jun 11, 2018 · 1 comment
Closed

Allow injecting logger or logger factory #190

xerial opened this issue Jun 11, 2018 · 1 comment

Comments

@xerial
Copy link
Member

xerial commented Jun 11, 2018

Currently Airframe uses the default logger (airframe-log). It might be useful if we can inject loggers (e.g., slf4j, scribe, etc.).

@xerial
Copy link
Member Author

xerial commented Jul 23, 2018

Injecting some LoggerFactory will be OK, but addtionally we need to pass the current class information to build a logger instance:

private val logger = {(cl : ThisClass, l:LoggerFactory) => LoggerFactory.of(cl) }

@xerial xerial added the idea label May 4, 2019
@xerial xerial mentioned this issue Dec 4, 2020
90 tasks
@xerial xerial added the wontfix label Dec 1, 2022
@xerial xerial closed this as completed Dec 1, 2022
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