-
Notifications
You must be signed in to change notification settings - Fork 818
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
其它的log日志都有Field操作,klog没有封装,导致用起来很不方便,还要使用底层的比如logrus.Field #1243
Comments
你这种场景直接用 logrus 就可以,klog 主要是为了打印框架日志,简单场景,你保证框架打印日志也能收集到即可。 |
我是从java转过来。目前在设计一个微服务整体结构。
|
cc @rogerogers |
目前 klog 的接口设计很简单,不能满足用户的需求,之前一直的解决方案都是复杂需求直接使用日志库,而不是框架的包装库,如果想满足需求的话现在看势必会有 break changg ,对用户和维护者造成一定的迁移用法压力。长期我们打算重构掉&统一 klog 和 hlog(hertz 的实现),短期的话我们看下具体场景有没有方案可以快速解决你的问题 @systemview2018 。 |
|
@systemview2018 “java SLF4j这的facade能力” 是指的占位符参数日志吗?这个目前的接口定义应该是有的 & 所有的实现库都支持,Golang 里是通过 format 的方式引入的这类能力,类似 Errorf(format string, v ...interface{}) 这种接口;如果可以的可以详细展开看看目前的日志抽象有哪些地方不能够满足当前的需求🫡 |
其实就是一个统一对外接口,只是这个接口包含的比较全面。 底层是可以其它的框架实现。https://blog.csdn.net/trh_csdn/article/details/127098291 |
其实引出这个问题也是想看看klog是不是有这个打算做一个通用的facade库,包含各个方面的。 确实可以直接使用具体的库,但是这个不是我的本意。 |
嗯嗯,看起来是对的上的,目前这套接口其实就是一个类似 facade 的作用,你可以看看上面提到的 formatLogger 的定义,这个也是 klog 的接口的一部分,看看是否除了 format(占位符)以外还有别的无法满足的诉求哈 |
@systemview2018 otel 这块的集成目前确实也存在一些 hertz & kitex 的 overlap,可观测相关生态缺少一个统一集成,这块我们正在开始努力解决哈:#1247 |
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Kitex version:
Please provide the version of Kitex you are using.
Environment:
The output of
go env
.Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: