services/tree: Move relaying code to a separate function
Some checks failed
DCO action / DCO (pull_request) Failing after 2m11s
Tests and linters / Run gofumpt (pull_request) Successful in 3m18s
Pre-commit hooks / Pre-commit (pull_request) Successful in 4m20s
Tests and linters / Lint (pull_request) Successful in 4m52s
Vulncheck / Vulncheck (pull_request) Successful in 4m50s
Build / Build Components (pull_request) Successful in 4m58s
Tests and linters / Staticcheck (pull_request) Successful in 4m50s
Tests and linters / gopls check (pull_request) Successful in 4m54s
Tests and linters / Tests (pull_request) Successful in 6m59s
Tests and linters / Tests with -race (pull_request) Successful in 6m59s
Some checks failed
DCO action / DCO (pull_request) Failing after 2m11s
Tests and linters / Run gofumpt (pull_request) Successful in 3m18s
Pre-commit hooks / Pre-commit (pull_request) Successful in 4m20s
Tests and linters / Lint (pull_request) Successful in 4m52s
Vulncheck / Vulncheck (pull_request) Successful in 4m50s
Build / Build Components (pull_request) Successful in 4m58s
Tests and linters / Staticcheck (pull_request) Successful in 4m50s
Tests and linters / gopls check (pull_request) Successful in 4m54s
Tests and linters / Tests (pull_request) Successful in 6m59s
Tests and linters / Tests with -race (pull_request) Successful in 6m59s
Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
parent
29708b78d7
commit
752de40ce4
2 changed files with 20 additions and 60 deletions
|
@ -12,10 +12,24 @@ import (
|
|||
"go.opentelemetry.io/otel/attribute"
|
||||
"go.opentelemetry.io/otel/trace"
|
||||
"go.uber.org/zap"
|
||||
"google.golang.org/grpc"
|
||||
)
|
||||
|
||||
var errNoSuitableNode = errors.New("no node was found to execute the request")
|
||||
|
||||
func relayUnary[Req any, Resp any](ctx context.Context, s *Service, ns []netmapSDK.NodeInfo, req *Req, callback func(TreeServiceClient, context.Context, *Req, ...grpc.CallOption) (*Resp, error)) (*Resp, error) {
|
||||
var resp *Resp
|
||||
var outErr error
|
||||
err := s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = callback(c, ctx, req)
|
||||
return true
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
}
|
||||
|
||||
// forEachNode executes callback for each node in the container until true is returned.
|
||||
// Returns errNoSuitableNode if there was no successful attempt to dial any node.
|
||||
func (s *Service) forEachNode(ctx context.Context, cntNodes []netmapSDK.NodeInfo, f func(c TreeServiceClient) bool) error {
|
||||
|
|
|
@ -122,16 +122,7 @@ func (s *Service) Add(ctx context.Context, req *AddRequest) (*AddResponse, error
|
|||
return nil, err
|
||||
}
|
||||
if pos < 0 {
|
||||
var resp *AddResponse
|
||||
var outErr error
|
||||
err = s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = c.Add(ctx, req)
|
||||
return true
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
return relayUnary(ctx, s, ns, req, (TreeServiceClient).Add)
|
||||
}
|
||||
|
||||
d := pilorama.CIDDescriptor{CID: cid, Position: pos, Size: len(ns)}
|
||||
|
@ -174,16 +165,7 @@ func (s *Service) AddByPath(ctx context.Context, req *AddByPathRequest) (*AddByP
|
|||
return nil, err
|
||||
}
|
||||
if pos < 0 {
|
||||
var resp *AddByPathResponse
|
||||
var outErr error
|
||||
err = s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = c.AddByPath(ctx, req)
|
||||
return true
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
return relayUnary(ctx, s, ns, req, (TreeServiceClient).AddByPath)
|
||||
}
|
||||
|
||||
meta := protoToMeta(b.GetMeta())
|
||||
|
@ -238,16 +220,7 @@ func (s *Service) Remove(ctx context.Context, req *RemoveRequest) (*RemoveRespon
|
|||
return nil, err
|
||||
}
|
||||
if pos < 0 {
|
||||
var resp *RemoveResponse
|
||||
var outErr error
|
||||
err = s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = c.Remove(ctx, req)
|
||||
return true
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
return relayUnary(ctx, s, ns, req, (TreeServiceClient).Remove)
|
||||
}
|
||||
|
||||
if b.GetNodeId() == pilorama.RootID {
|
||||
|
@ -291,16 +264,7 @@ func (s *Service) Move(ctx context.Context, req *MoveRequest) (*MoveResponse, er
|
|||
return nil, err
|
||||
}
|
||||
if pos < 0 {
|
||||
var resp *MoveResponse
|
||||
var outErr error
|
||||
err = s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = c.Move(ctx, req)
|
||||
return true
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
return relayUnary(ctx, s, ns, req, (TreeServiceClient).Move)
|
||||
}
|
||||
|
||||
if b.GetNodeId() == pilorama.RootID {
|
||||
|
@ -343,16 +307,7 @@ func (s *Service) GetNodeByPath(ctx context.Context, req *GetNodeByPathRequest)
|
|||
return nil, err
|
||||
}
|
||||
if pos < 0 {
|
||||
var resp *GetNodeByPathResponse
|
||||
var outErr error
|
||||
err = s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = c.GetNodeByPath(ctx, req)
|
||||
return true
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
return relayUnary(ctx, s, ns, req, (TreeServiceClient).GetNodeByPath)
|
||||
}
|
||||
|
||||
attr := b.GetPathAttribute()
|
||||
|
@ -763,16 +718,7 @@ func (s *Service) TreeList(ctx context.Context, req *TreeListRequest) (*TreeList
|
|||
return nil, err
|
||||
}
|
||||
if pos < 0 {
|
||||
var resp *TreeListResponse
|
||||
var outErr error
|
||||
err = s.forEachNode(ctx, ns, func(c TreeServiceClient) bool {
|
||||
resp, outErr = c.TreeList(ctx, req)
|
||||
return outErr == nil
|
||||
})
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
return resp, outErr
|
||||
return relayUnary(ctx, s, ns, req, (TreeServiceClient).TreeList)
|
||||
}
|
||||
|
||||
ids, err := s.forest.TreeList(ctx, cid)
|
||||
|
|
Loading…
Reference in a new issue