Skip to content

Commit 98c975b

Browse files
wenyihu6tbg
authored andcommitted
asim: remove unused return value in checkPendingTicket
This commit removes an unused return value from checkPendingTicket(), as it is not used anywhere. Epic: none Release note: none
1 parent e66e2be commit 98c975b

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

pkg/kv/kvserver/asim/storerebalancer/store_rebalancer.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -139,17 +139,17 @@ func (src *storeRebalancerControl) scorerOptions() *allocatorimpl.LoadScorerOpti
139139
}
140140
}
141141

142-
func (src *storeRebalancerControl) checkPendingTicket() (done bool, next time.Time, _ error) {
142+
func (src *storeRebalancerControl) checkPendingTicket() (done bool, _ error) {
143143
ticket := src.rebalancerState.pendingTicket
144144
op, ok := src.controller.Check(ticket)
145145
if !ok {
146-
return true, time.Time{}, nil
146+
return true, nil
147147
}
148-
done, next = op.Done()
148+
done, _ = op.Done()
149149
if !done {
150-
return false, op.Next(), nil
150+
return false, nil
151151
}
152-
return true, next, op.Errors()
152+
return true, op.Errors()
153153
}
154154

155155
func (src *storeRebalancerControl) Tick(ctx context.Context, tick time.Time, state state.State) {
@@ -207,7 +207,7 @@ func (src *storeRebalancerControl) checkPendingLeaseRebalance(ctx context.Contex
207207
return true
208208
}
209209

210-
done, _, err := src.checkPendingTicket()
210+
done, err := src.checkPendingTicket()
211211
if !done {
212212
// No more we can do in this tick - we need to wait for the
213213
// transfer to complete.
@@ -299,7 +299,7 @@ func (src *storeRebalancerControl) checkPendingRangeRebalance(ctx context.Contex
299299
return true
300300
}
301301

302-
done, _, err := src.checkPendingTicket()
302+
done, err := src.checkPendingTicket()
303303
if !done {
304304
// No more we can do in this tick - we need to wait for the
305305
// relocate to complete.

0 commit comments

Comments
 (0)