Skip to content

Commit

Permalink
Merge pull request #1 from apache/master
Browse files Browse the repository at this point in the history
fix the bug caused in HA cluster in pull_consumer (apache#717)
  • Loading branch information
yanjianlong authored Aug 24, 2021
2 parents 0442864 + 78f607d commit 803b0b1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion consumer/pull_consumer.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ func NewPullConsumer(options ...Option) (*defaultPullConsumer, error) {

namesrv: srvs,
}

dc.option.ClientOptions.Namesrv, err = internal.GetNamesrv(dc.client.ClientID())
c := &defaultPullConsumer{
defaultConsumer: dc,
}
Expand Down
1 change: 1 addition & 0 deletions internal/route.go
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,7 @@ func (s *namesrvs) UpdateTopicRouteInfoWithDefault(topic string, defaultTopic st

return true
})
rlog.Info("change the route for clients", nil)
}

s.routeDataMap.Store(topic, routeData)
Expand Down

0 comments on commit 803b0b1

Please sign in to comment.