You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by GitBox <gi...@apache.org> on 2020/12/13 06:46:03 UTC

[GitHub] [dubbo-go] cvictory commented on a change in pull request #915: fix: health check

cvictory commented on a change in pull request #915:
URL: https://github.com/apache/dubbo-go/pull/915#discussion_r541860297



##########
File path: config/reference_config.go
##########
@@ -41,30 +41,31 @@ import (
 
 // ReferenceConfig is the configuration of service consumer
 type ReferenceConfig struct {
-	context        context.Context
-	pxy            *proxy.Proxy
-	id             string
-	InterfaceName  string            `required:"true"  yaml:"interface"  json:"interface,omitempty" property:"interface"`
-	Check          *bool             `yaml:"check"  json:"check,omitempty" property:"check"`
-	Url            string            `yaml:"url"  json:"url,omitempty" property:"url"`
-	Filter         string            `yaml:"filter" json:"filter,omitempty" property:"filter"`
-	Protocol       string            `default:"dubbo"  yaml:"protocol"  json:"protocol,omitempty" property:"protocol"`
-	Registry       string            `yaml:"registry"  json:"registry,omitempty"  property:"registry"`
-	Cluster        string            `yaml:"cluster"  json:"cluster,omitempty" property:"cluster"`
-	Loadbalance    string            `yaml:"loadbalance"  json:"loadbalance,omitempty" property:"loadbalance"`
-	Retries        string            `yaml:"retries"  json:"retries,omitempty" property:"retries"`
-	Group          string            `yaml:"group"  json:"group,omitempty" property:"group"`
-	Version        string            `yaml:"version"  json:"version,omitempty" property:"version"`
-	ProvideBy      string            `yaml:"provide_by"  json:"provide_by,omitempty" property:"provide_by"`
-	Methods        []*MethodConfig   `yaml:"methods"  json:"methods,omitempty" property:"methods"`
-	Async          bool              `yaml:"async"  json:"async,omitempty" property:"async"`
-	Params         map[string]string `yaml:"params"  json:"params,omitempty" property:"params"`
-	invoker        protocol.Invoker
-	urls           []*common.URL
-	Generic        bool   `yaml:"generic"  json:"generic,omitempty" property:"generic"`
-	Sticky         bool   `yaml:"sticky"   json:"sticky,omitempty" property:"sticky"`
-	RequestTimeout string `yaml:"timeout"  json:"timeout,omitempty" property:"timeout"`
-	ForceTag       bool   `yaml:"force.tag"  json:"force.tag,omitempty" property:"force.tag"`
+	context           context.Context
+	pxy               *proxy.Proxy
+	id                string
+	InterfaceName     string            `required:"true"  yaml:"interface"  json:"interface,omitempty" property:"interface"`
+	Check             *bool             `yaml:"check"  json:"check,omitempty" property:"check"`
+	Url               string            `yaml:"url"  json:"url,omitempty" property:"url"`
+	Filter            string            `yaml:"filter" json:"filter,omitempty" property:"filter"`
+	Protocol          string            `default:"dubbo"  yaml:"protocol"  json:"protocol,omitempty" property:"protocol"`
+	Registry          string            `yaml:"registry"  json:"registry,omitempty"  property:"registry"`
+	Cluster           string            `yaml:"cluster"  json:"cluster,omitempty" property:"cluster"`
+	Loadbalance       string            `yaml:"loadbalance"  json:"loadbalance,omitempty" property:"loadbalance"`
+	Retries           string            `yaml:"retries"  json:"retries,omitempty" property:"retries"`
+	Group             string            `yaml:"group"  json:"group,omitempty" property:"group"`
+	Version           string            `yaml:"version"  json:"version,omitempty" property:"version"`
+	ProvideBy         string            `yaml:"provide_by"  json:"provide_by,omitempty" property:"provide_by"`
+	Methods           []*MethodConfig   `yaml:"methods"  json:"methods,omitempty" property:"methods"`
+	Async             bool              `yaml:"async"  json:"async,omitempty" property:"async"`
+	Params            map[string]string `yaml:"params"  json:"params,omitempty" property:"params"`
+	invoker           protocol.Invoker
+	urls              []*common.URL
+	Generic           bool   `yaml:"generic"  json:"generic,omitempty" property:"generic"`
+	Sticky            bool   `yaml:"sticky"   json:"sticky,omitempty" property:"sticky"`
+	RequestTimeout    string `yaml:"timeout"  json:"timeout,omitempty" property:"timeout"`
+	ForceTag          bool   `yaml:"force.tag"  json:"force.tag,omitempty" property:"force.tag"`
+	HealthCheckEnable bool   `yaml:"health_check" json:"health_check,omitempty" property:"health_check"`

Review comment:
       这个放到Params里有什么问题?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org