Skip to content

Commit

Permalink
server, client: fix hanging problem when etcd failed to start (tikv#1267
Browse files Browse the repository at this point in the history
)
  • Loading branch information
disksing committed Nov 14, 2018
1 parent b647167 commit 5f7712d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
4 changes: 3 additions & 1 deletion pd-client/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,9 @@ func (c *client) initClusterID() error {
defer cancel()
for i := 0; i < maxInitClusterRetries; i++ {
for _, u := range c.urls {
members, err := c.getMembers(ctx, u)
timeoutCtx, timeoutCancel := context.WithTimeout(ctx, pdTimeout)
members, err := c.getMembers(timeoutCtx, u)
timeoutCancel()
if err != nil || members.GetHeader() == nil {
log.Errorf("[pd] failed to get cluster id: %v", err)
continue
Expand Down
6 changes: 5 additions & 1 deletion server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ import (
)

const (
etcdTimeout = time.Second * 3
etcdTimeout = time.Second * 3
etcdStartTimeout = time.Minute * 5
// pdRootPath for all pd servers.
pdRootPath = "/pd"
pdAPIPrefix = "/pd/"
Expand Down Expand Up @@ -120,6 +121,9 @@ func CreateServer(cfg *Config, apiRegister func(*Server) http.Handler) (*Server,

func (s *Server) startEtcd() error {
log.Info("start embed etcd")
ctx, cancel := context.WithTimeout(ctx, etcdStartTimeout)
defer cancel()

etcd, err := embed.StartEtcd(s.etcdCfg)
if err != nil {
return errors.Trace(err)
Expand Down

0 comments on commit 5f7712d

Please sign in to comment.