Skip to content

Commit a9ce335

Browse files
committed
fixed linter warnings
1 parent 1825a9e commit a9ce335

File tree

5 files changed

+8
-10
lines changed

5 files changed

+8
-10
lines changed

internal/background/worker_test.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,6 @@ func TestWorkerConcurrentStartAndClose(t *testing.T) {
106106
ctx := xtest.Context(t)
107107
w := NewWorker(ctx)
108108

109-
closeIndex := int64(0)
110-
111109
stopNewStarts := xatomic.Bool{}
112110
var wgStarters sync.WaitGroup
113111
for i := 0; i < parallel; i++ {
@@ -134,11 +132,12 @@ func TestWorkerConcurrentStartAndClose(t *testing.T) {
134132

135133
require.NoError(t, w.Close(xtest.ContextWithCommonTimeout(ctx, t), nil))
136134

137-
closeIndex = counter.Load()
138-
require.Equal(t, closeIndex, counter.Load())
139-
140135
stopNewStarts.Store(true)
141136
xtest.WaitGroup(t, &wgStarters)
137+
138+
_, ok := <-w.tasks
139+
require.False(t, ok)
140+
require.True(t, w.closed)
142141
})
143142
}
144143

internal/table/scanner/perfomance_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ func BenchmarkTestSliceIncrement(b *testing.B) {
123123
for j := 0; j < testSize; j++ {
124124
slice[j] = &column{}
125125
}
126-
cnt := 0
126+
var cnt int
127127
var row column
128128
b.ResetTimer()
129129
for i := 0; i < b.N; i++ {

internal/topic/retriable_error.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,6 @@ func CheckRetryMode(err error, settings RetrySettings, retriesDuration time.Dura
6565
return nil, false
6666
}
6767

68-
isRetriable = true
69-
7068
mode := retry.Check(err)
7169

7270
decision := PublicRetryDecisionDefault

internal/topic/topicwriterinternal/writer_reconnector.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -351,7 +351,7 @@ func (w *WriterReconnector) connectionLoop(ctx context.Context) {
351351
return xcontext.WithCancel(xcontext.WithoutDeadline(ctx))
352352
}
353353

354-
//nolint:ineffassign,staticcheck
354+
//nolint:ineffassign,staticcheck,wastedassign
355355
streamCtx, streamCtxCancel := createStreamContext()
356356

357357
defer streamCtxCancel()

table/options/models.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,8 @@ func (rr ReadReplicasSettings) ToYDB() *Ydb_Table.ReadReplicasSettings {
198198
}
199199

200200
func NewReadReplicasSettings(rr *Ydb_Table.ReadReplicasSettings) ReadReplicasSettings {
201-
t, c := ReadReplicasPerAzReadReplicas, uint64(0)
201+
t := ReadReplicasPerAzReadReplicas
202+
var c uint64
202203

203204
if c = rr.GetPerAzReadReplicasCount(); c != 0 {
204205
t = ReadReplicasPerAzReadReplicas

0 commit comments

Comments
 (0)