Alanxtl commented on code in PR #3346:
URL: https://github.com/apache/dubbo-go/pull/3346#discussion_r3329662305
##########
protocol/protocolwrapper/protocol_filter_wrapper.go:
##########
@@ -92,10 +92,10 @@ func BuildInvokerChain(invoker base.Invoker, key string)
base.Invoker {
}
switch key {
case constant.ServiceFilterKey:
- logger.Debugf("[BuildInvokerChain] The provider invocation link
is %s, invoker: %s",
+ logger.Debugf("[ProtocolWrapper] The provider invocation link
is %s, invoker: %s",
strings.Join(append(filterNames, "proxyInvoker"), " ->
"), invoker)
case constant.ReferenceFilterKey:
- logger.Debugf("[BuildInvokerChain] The consumer filters are %s,
invoker: %s",
+ logger.Debugf("[ProtocolWrapper] The consumer filters are %s,
invoker: %s",
Review Comment:
```suggestion
logger.Debugf("[Protocol][Wrapper] The provider invocation link
is %s, invoker: %s",
strings.Join(append(filterNames, "proxyInvoker"), " -> "), invoker)
case constant.ReferenceFilterKey:
logger.Debugf("[Protocol][Wrapper] The consumer filters are %s,
invoker: %s",
```
##########
options.go:
##########
@@ -93,8 +93,8 @@ func (rc *InstanceOptions) init(opts ...InstanceOption) error
{
return err
}
if loadedRemoteConfig, err := rc.initGlobalConfigCenter(); err != nil {
- logger.Infof("[Config Center] Config center doesn't start")
- logger.Debugf("config center doesn't start because %s", err)
+ logger.Info("[Config Center] config center doesn't start")
+ logger.Debugf("[Config Center] config center doesn't start,
err=%v", err)
Review Comment:
```suggestion
logger.Info("[ConfigCenter] config center doesn't start")
logger.Debugf("[ConfigCenter] config center doesn't start,
err=%v", err)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
