From 529d0bc710e72baed1e5523117d061e2b01ffe16 Mon Sep 17 00:00:00 2001 From: Dmitrii Stepanov Date: Tue, 2 May 2023 16:50:03 +0300 Subject: [PATCH] [#302] tree: Drop unused ctx Signed-off-by: Dmitrii Stepanov --- pkg/services/tree/sync.go | 4 ++-- pkg/services/tree/sync_test.go | 7 +------ 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/pkg/services/tree/sync.go b/pkg/services/tree/sync.go index 5c0300c14..ed2455194 100644 --- a/pkg/services/tree/sync.go +++ b/pkg/services/tree/sync.go @@ -126,7 +126,7 @@ func (s *Service) SynchronizeTree(ctx context.Context, cid cid.ID, treeID string } // mergeOperationStreams performs merge sort for node operation streams to one stream. -func mergeOperationStreams(ctx context.Context, streams []chan *pilorama.Move, merged chan<- *pilorama.Move) uint64 { +func mergeOperationStreams(streams []chan *pilorama.Move, merged chan<- *pilorama.Move) uint64 { defer close(merged) ms := make([]*pilorama.Move, len(streams)) @@ -274,7 +274,7 @@ func (s *Service) synchronizeTree(ctx context.Context, cid cid.ID, from uint64, merged := make(chan *pilorama.Move) var minStreamedLastHeight uint64 errGroup.Go(func() error { - minStreamedLastHeight = mergeOperationStreams(ctx, nodeOperationStreams, merged) + minStreamedLastHeight = mergeOperationStreams(nodeOperationStreams, merged) return nil }) var minUnappliedHeight uint64 diff --git a/pkg/services/tree/sync_test.go b/pkg/services/tree/sync_test.go index eb51bc3f8..190b4ccbb 100644 --- a/pkg/services/tree/sync_test.go +++ b/pkg/services/tree/sync_test.go @@ -1,7 +1,6 @@ package tree import ( - "context" "testing" "git.frostfs.info/TrueCloudLab/frostfs-node/pkg/local_object_storage/pilorama" @@ -11,14 +10,12 @@ import ( func Test_mergeOperationStreams(t *testing.T) { tests := []struct { name string - ctx context.Context opTimes [][]uint64 wantValues []uint64 wantMinHeight uint64 }{ { name: "1", - ctx: context.Background(), opTimes: [][]uint64{ {250, 251, 255}, {252, 253, 254, 256, 257}, @@ -28,7 +25,6 @@ func Test_mergeOperationStreams(t *testing.T) { }, { name: "2", - ctx: context.Background(), opTimes: [][]uint64{ {250, 251, 255, 259}, {252, 253, 254, 256, 257}, @@ -38,7 +34,6 @@ func Test_mergeOperationStreams(t *testing.T) { }, { name: "3", - ctx: context.Background(), opTimes: [][]uint64{ {250, 251, 255}, {249, 250, 251, 253, 254, 256, 257}, @@ -71,7 +66,7 @@ func Test_mergeOperationStreams(t *testing.T) { merged := make(chan *pilorama.Move, 1) min := make(chan uint64) go func() { - min <- mergeOperationStreams(tt.ctx, nodeOpChans, merged) + min <- mergeOperationStreams(nodeOpChans, merged) }() var res []uint64