dubbo-go
dubbo-go copied to clipboard
日志初始化的时候,希望能规范stdout与stderr,,同时也防止log重复初始化
dubbo.apache.org/dubbo-go/[email protected]/common/logger 中 初始化config.ZapConfig函数希望能够将 OutputPaths: []string{"stderr"}, 修改为 OutputPaths: []string{"stdout"},
因为每次初始化的时候,都会重新初始化log,导致监控报警这部分出现误报 同时也希望在初始化日志的时候,首先判断一下log有无初始化,这样能够防止log重复初始化
good suggestion.
https://github.com/dubbogo/dubbo-go-boot @cjphaha 可以看下这个里面的代码整理下
assign https://github.com/codingadai
OutputPaths的默认值应该是源自zap,改为stdout,是否要考虑先后兼容性?这个在zap有个issue讨论过。 @zhaoyunxing92
logger初始化的地方有3处:
- github.com/dubbogo/[email protected]/log/logger/logger.go:29 (init函数里,这块会最先执行)
- dubbo-go/config/root_config.go:127 (根据配置文件初始化logger)
- dubbo-go/config/root_config.go:134 (如果有config-center配置,会从config-center中重新加载配置再初始化一次) 最主要是gost logger init函数和 dubbo-go中的 这次重复,我看下怎么改进更合适。