Skip to content

Commit c738a82

Browse files
committed
Remove redundant TODO comment on processLockedKeys
1 parent 7f0145f commit c738a82

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

services/requester/keystore/key_store.go

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -172,19 +172,6 @@ func (k *KeyStore) processLockedKeys(ctx context.Context) {
172172
close(k.done)
173173
return
174174
case blockHeader := <-k.blockChan:
175-
// TODO: If calling `k.client.GetTransactionResultsByBlockID` for each
176-
// new block, seems to be problematic for ANs, we can take an approach
177-
// like the one below:
178-
// If the available keys ratio is >= 60% of the total signing keys,
179-
// then we can skip checking the transaction result statuses.
180-
// The signing keys from invalid EVM transactions, will eventually
181-
// come again into availability, after `accountKeyBlockExpiration`
182-
// blocks have passed.
183-
// availablekeysRatio := float64(k.AvailableKeys()) / float64(k.size)
184-
// if availablekeysRatio >= 0.6 {
185-
// continue
186-
// }
187-
188175
// Optimization to avoid AN calls when no signing keys have
189176
// been used. For example, when back-filling the EVM GW state,
190177
// we don't care about releasing signing keys.

0 commit comments

Comments
 (0)