From 9ce67d2147a91073402c340a8188bb394164bb73 Mon Sep 17 00:00:00 2001 From: colin <102356659+colinlyguo@users.noreply.github.com> Date: Tue, 17 Sep 2024 22:03:15 +0800 Subject: [PATCH] feat(scroll-worker): add nil consumption block metrics (#1033) * feat(scroll-worker): add nil consumption block metrics * increase the version --- miner/scroll_worker.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/miner/scroll_worker.go b/miner/scroll_worker.go index 3d01f6e9bf40..4ff005f8b58e 100644 --- a/miner/scroll_worker.go +++ b/miner/scroll_worker.go @@ -72,6 +72,9 @@ var ( commitReasonCCCCounter = metrics.NewRegisteredCounter("miner/commit_reason_ccc", nil) commitReasonDeadlineCounter = metrics.NewRegisteredCounter("miner/commit_reason_deadline", nil) commitGasCounter = metrics.NewRegisteredCounter("miner/commit_gas", nil) + + missingRCOnRestartCounter = metrics.NewRegisteredCounter("miner/missing_rc_on_restart", nil) + missingAncestorRCCounter = metrics.NewRegisteredCounter("miner/missing_ancestor_rc", nil) ) // prioritizedTransaction represents a single transaction that @@ -240,6 +243,8 @@ func (w *worker) checkHeadRowConsumption() error { block := w.chain.GetBlockByNumber(curBlockNum) // only spawn CCC checkers for blocks with no row consumption data stored in DB if rawdb.ReadBlockRowConsumption(w.chain.Database(), block.Hash()) == nil { + missingRCOnRestartCounter.Inc(1) + if err := w.asyncChecker.Check(block); err != nil { return err } @@ -812,6 +817,8 @@ func (w *worker) commit(reorging bool) (common.Hash, error) { ancestorHeight := currentHeight - maxReorgDepth ancestorHash := w.chain.GetHeaderByNumber(ancestorHeight).Hash() if rawdb.ReadBlockRowConsumption(w.chain.Database(), ancestorHash) == nil { + missingAncestorRCCounter.Inc(1) + // reject committing to a block if its ancestor doesn't have its RC stored in DB yet. // which may either mean that it failed CCC or it is still in the process of being checked return common.Hash{}, retryableCommitError{inner: errors.New("ancestor doesn't have RC yet")}