[#353] pool: Remove deprecated IterateNetworkEndpoints()
Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
parent
a0e4d16dbb
commit
f158a6b2e1
1 changed files with 7 additions and 17 deletions
|
@ -1081,20 +1081,15 @@ func (p *Pool) deleteClientFromMap(hash uint64) {
|
||||||
}
|
}
|
||||||
|
|
||||||
func (p *Pool) getNewTreeClient(ctx context.Context, node netmap.NodeInfo) (*treeClient, error) {
|
func (p *Pool) getNewTreeClient(ctx context.Context, node netmap.NodeInfo) (*treeClient, error) {
|
||||||
var (
|
for endpoint := range node.NetworkEndpoints() {
|
||||||
treeCl *treeClient
|
|
||||||
err error
|
|
||||||
)
|
|
||||||
|
|
||||||
node.IterateNetworkEndpoints(func(endpoint string) bool {
|
|
||||||
var addr network.Address
|
var addr network.Address
|
||||||
if err = addr.FromString(endpoint); err != nil {
|
if err := addr.FromString(endpoint); err != nil {
|
||||||
p.log(zap.WarnLevel, "can't parse endpoint", zap.String("endpoint", endpoint), zap.Error(err))
|
p.log(zap.WarnLevel, "can't parse endpoint", zap.String("endpoint", endpoint), zap.Error(err))
|
||||||
return false
|
continue
|
||||||
}
|
}
|
||||||
|
|
||||||
newTreeCl := newTreeClient(addr.URIAddr(), p.dialOptions, p.nodeDialTimeout, p.streamTimeout)
|
newTreeCl := newTreeClient(addr.URIAddr(), p.dialOptions, p.nodeDialTimeout, p.streamTimeout)
|
||||||
if err = newTreeCl.dial(ctx); err != nil {
|
if err := newTreeCl.dial(ctx); err != nil {
|
||||||
p.log(zap.WarnLevel, "failed to dial tree client", zap.Error(err))
|
p.log(zap.WarnLevel, "failed to dial tree client", zap.Error(err))
|
||||||
|
|
||||||
// We have to close connection here after failed `dial()`.
|
// We have to close connection here after failed `dial()`.
|
||||||
|
@ -1106,18 +1101,13 @@ func (p *Pool) getNewTreeClient(ctx context.Context, node netmap.NodeInfo) (*tre
|
||||||
p.log(zap.WarnLevel, "failed to close recently dialed tree client", zap.Error(err))
|
p.log(zap.WarnLevel, "failed to close recently dialed tree client", zap.Error(err))
|
||||||
}
|
}
|
||||||
|
|
||||||
return false
|
continue
|
||||||
}
|
}
|
||||||
|
|
||||||
treeCl = newTreeCl
|
return newTreeCl, nil
|
||||||
return true
|
|
||||||
})
|
|
||||||
|
|
||||||
if treeCl == nil {
|
|
||||||
return nil, fmt.Errorf("tree client wasn't initialized")
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return treeCl, nil
|
return nil, fmt.Errorf("tree client wasn't initialized")
|
||||||
}
|
}
|
||||||
|
|
||||||
func shouldTryAgain(err error) bool {
|
func shouldTryAgain(err error) bool {
|
||||||
|
|
Loading…
Add table
Reference in a new issue