[#741] treesvc: Remove unused height variables

Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>

internal/logs/logs.go

@@ -64,6 +64,7 @@ const (
 	TreeCouldNotUpdateLastSynchronizedHeightForATree                        = "could not update last synchronized height for a tree"
 	TreeSynchronizeTree                                                     = "synchronize tree"
 	TreeFailedToRunTreeSynchronizationOverAllNodes                          = "failed to run tree synchronization over all nodes"
+	TreeFailedToRunTreeSynchronizationForSpecificNode                       = "failed to run tree synchronization for specific node"
 	TreeSyncingTrees                                                        = "syncing trees..."
 	TreeCouldNotFetchContainers                                             = "could not fetch containers"
 	TreeTreesHaveBeenSynchronized                                           = "trees have been synchronized"

pkg/services/tree/sync.go

@@ -217,26 +217,24 @@ func (s *Service) applyOperationStream(ctx context.Context, cid cid.ID, treeID s
 
 func (s *Service) startStream(ctx context.Context, cid cid.ID, treeID string,
 	height uint64, treeClient TreeServiceClient, opsCh chan<- *pilorama.Move,
-) (uint64, error) {
+) error {
 	rawCID := make([]byte, sha256.Size)
 	cid.Encode(rawCID)
 
-	for {
-		newHeight := height
 	req := &GetOpLogRequest{
 		Body: &GetOpLogRequest_Body{
 			ContainerId: rawCID,
 			TreeId:      treeID,
-				Height:      newHeight,
+			Height:      height,
 		},
 	}
 	if err := SignMessage(req, s.key); err != nil {
-			return 0, err
+		return err
 	}
 
 	c, err := treeClient.GetOpLog(ctx, req)
 	if err != nil {
-			return 0, fmt.Errorf("can't initialize client: %w", err)
+		return fmt.Errorf("can't initialize client: %w", err)
 	}
 	res, err := c.Recv()
 	for ; err == nil; res, err = c.Recv() {
@@ -246,15 +244,14 @@ func (s *Service) startStream(ctx context.Context, cid cid.ID, treeID string,
 			Child:  lm.ChildId,
 		}
 		if err := m.Meta.FromBytes(lm.Meta); err != nil {
-				return 0, err
+			return err
 		}
 		opsCh <- m
 	}
-		if height == newHeight || err != nil && !errors.Is(err, io.EOF) {
+	if err != nil && !errors.Is(err, io.EOF) {
-			return newHeight, err
+		return err
-		}
-		height = newHeight
 	}
+	return nil
 }
 
 // synchronizeTree synchronizes operations getting them from different nodes.
@@ -291,7 +288,6 @@ func (s *Service) synchronizeTree(ctx context.Context, cid cid.ID, from uint64,
 		i := i
 		n := n
 		errGroup.Go(func() error {
-			height := from
 			n.IterateNetworkEndpoints(func(addr string) bool {
 				var a network.Address
 				if err := a.FromString(addr); err != nil {
@@ -315,16 +311,12 @@ func (s *Service) synchronizeTree(ctx context.Context, cid cid.ID, from uint64,
 				defer cc.Close()
 
 				treeClient := NewTreeServiceClient(cc)
-				for {
+				err = s.startStream(egCtx, cid, treeID, from, treeClient, nodeOperationStreams[i])
-					h, err := s.startStream(egCtx, cid, treeID, from, treeClient, nodeOperationStreams[i])
+				if err != nil {
-					if height < h {
-						height = h
-					}
-					if err != nil || h <= height {
 					// Error with the response, try the next node.
+					s.log.Warn(logs.TreeFailedToRunTreeSynchronizationForSpecificNode, zap.Error(err), zap.String("address", addr))
+				}
 				return true
-					}
-				}
 			})
 			close(nodeOperationStreams[i])
 			return nil