Skip to content

perf(log): 优化debug日志输出性能 #369

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EquentR
Copy link
Contributor

@EquentR EquentR commented Feb 28, 2025

  • 在调试模式下条件性地输出 debug 日志,减少不必要的日志打印

issue:#366

- 在调试模式下条件性地输出 debug 日志,减少不必要的日志打印
@aceld
Copy link
Owner

aceld commented Feb 28, 2025

@EquentR 感谢提交PR!

@aceld
Copy link
Owner

aceld commented Feb 28, 2025

@EquentR 看了下,修复的问题和思路没问题,我之前欠考虑了,主要问题抽象层ILogger() 增加了接口,确实存在无法向下兼容的问题,可能会导致升级用户编译的问题,这块看看如何做到用户无感知,或者兼容写法,容我再思考思考哈~

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.

None yet

2 participants