bugfix: removing measuring of metrics from async batch reprocessing f… (

#2461) * bugfix: removing measuring of metrics from async batch reprocessing for executor. Signed-off-by: Nikolay Nedkov <[email protected]> * fixing unit tests Signed-off-by: Nikolay Nedkov <[email protected]> --------- Signed-off-by: Nikolay Nedkov <[email protected]>
0xPolygonHermez · Aug 28, 2023 · 307e83e · 307e83e
1 parent f16d9ae
commit 307e83e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/sequencer/finalizer.go b/sequencer/finalizer.go
@@ -1113,15 +1113,18 @@ func (f *finalizer) reprocessFullBatch(ctx context.Context, batchNum uint64, ini
 	}
 
 	log.Infof("reprocessFullBatch: BatchNumber: %d, OldStateRoot: %s, ExpectedNewStateRoot: %s, GER: %s", batch.BatchNumber, initialStateRoot.String(), expectedNewStateRoot.String(), batch.GlobalExitRoot.String())
-
+	caller := stateMetrics.DiscardCallerLabel
+	if f.cfg.SequentialReprocessFullBatch {
+		caller = stateMetrics.SequencerCallerLabel
+	}
 	processRequest := state.ProcessRequest{
 		BatchNumber:    batch.BatchNumber,
 		GlobalExitRoot: batch.GlobalExitRoot,
 		OldStateRoot:   initialStateRoot,
 		Transactions:   batch.BatchL2Data,
 		Coinbase:       batch.Coinbase,
 		Timestamp:      batch.Timestamp,
-		Caller:         stateMetrics.SequencerCallerLabel,
+		Caller:         caller,
 	}
 
 	forkID := f.dbManager.GetForkIDByBatchNumber(batchNum)

diff --git a/sequencer/finalizer_test.go b/sequencer/finalizer_test.go
@@ -81,6 +81,7 @@ var (
 		},
 		ResourcePercentageToCloseBatch: 10,
 		GERFinalityNumberOfBlocks:      64,
+		SequentialReprocessFullBatch:   true,
 	}
 	chainID         = new(big.Int).SetInt64(400)
 	pvtKey          = "0x28b2b0318721be8c8339199172cd7cc8f5e273800a35616ec893083a4b32c02e"