Skip to content

Commit dd8ff97

Browse files
committed
refactor: get bitmap for disabled verifiers to only do one time per batch
1 parent 6d96950 commit dd8ff97

File tree

1 file changed

+18
-9
lines changed

1 file changed

+18
-9
lines changed

operator/pkg/operator.go

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88
"encoding/json"
99
"fmt"
1010
"log"
11+
"math/big"
1112
"os"
1213
"sync"
1314
"time"
@@ -347,10 +348,18 @@ func (o *Operator) ProcessNewBatchLogV2(newBatchLog *servicemanager.ContractAlig
347348
results := make(chan bool, verificationDataBatchLen)
348349
var wg sync.WaitGroup
349350
wg.Add(verificationDataBatchLen)
351+
352+
disabledVerifiersBitmap, err := o.avsReader.DisabledVerifiers()
353+
if err != nil {
354+
o.Logger.Errorf("Could not check verifiers status: %s", err)
355+
results <- false
356+
return err
357+
}
358+
350359
for _, verificationData := range verificationDataBatch {
351360
go func(data VerificationData) {
352361
defer wg.Done()
353-
o.verify(data, results)
362+
o.verify(data, disabledVerifiersBitmap, results)
354363
o.metrics.IncOperatorTaskResponses()
355364
}(verificationData)
356365
}
@@ -420,10 +429,16 @@ func (o *Operator) ProcessNewBatchLogV3(newBatchLog *servicemanager.ContractAlig
420429
results := make(chan bool, verificationDataBatchLen)
421430
var wg sync.WaitGroup
422431
wg.Add(verificationDataBatchLen)
432+
disabledVerifiersBitmap, err := o.avsReader.DisabledVerifiers()
433+
if err != nil {
434+
o.Logger.Errorf("Could not check verifiers status: %s", err)
435+
results <- false
436+
return err
437+
}
423438
for _, verificationData := range verificationDataBatch {
424439
go func(data VerificationData) {
425440
defer wg.Done()
426-
o.verify(data, results)
441+
o.verify(data, disabledVerifiersBitmap, results)
427442
o.metrics.IncOperatorTaskResponses()
428443
}(verificationData)
429444
}
@@ -454,13 +469,7 @@ func (o *Operator) afterHandlingBatchV3(log *servicemanager.ContractAlignedLayer
454469
}
455470
}
456471

457-
func (o *Operator) verify(verificationData VerificationData, results chan bool) {
458-
disabledVerifiersBitmap, err := o.avsReader.DisabledVerifiers()
459-
if err != nil {
460-
o.Logger.Errorf("Could not check verifier status: %s", err)
461-
results <- false
462-
return
463-
}
472+
func (o *Operator) verify(verificationData VerificationData, disabledVerifiersBitmap *big.Int, results chan bool) {
464473
IsVerifierDisabled := IsVerifierDisabled(disabledVerifiersBitmap, verificationData.ProvingSystemId)
465474
if IsVerifierDisabled {
466475
o.Logger.Infof("Verifier %s is disbled. Skipping verifie", verificationData.ProvingSystemId.String())

0 commit comments

Comments
 (0)