feat(engine): more engine execution tracing spans (#20135)

Author: shekhirin

crates/engine/tree/src/tree/metrics.rs

@@ -79,7 +79,10 @@ impl EngineApiMetrics {
         let mut executor = executor.with_state_hook(Some(Box::new(wrapper)));
 
         let f = || {
-            executor.apply_pre_execution_changes()?;
+            debug_span!(target: "engine::tree", "pre execution")
+                .entered()
+                .in_scope(|| executor.apply_pre_execution_changes())?;
+            let exec_span = debug_span!(target: "engine::tree", "execution").entered();
             for tx in transactions {
                 let tx = tx?;
                 let span =
@@ -92,7 +95,11 @@ impl EngineApiMetrics {
                 // record the tx gas used
                 enter.record("gas_used", gas_used);
             }
-            executor.finish().map(|(evm, result)| (evm.into_db(), result))
+            drop(exec_span);
+            debug_span!(target: "engine::tree", "finish")
+                .entered()
+                .in_scope(|| executor.finish())
+                .map(|(evm, result)| (evm.into_db(), result))
         };
 
         // Use metered to execute and track timing/gas metrics
@@ -103,7 +110,9 @@ impl EngineApiMetrics {
         })?;
 
         // merge transitions into bundle state
-        db.borrow_mut().merge_transitions(BundleRetention::Reverts);
+        debug_span!(target: "engine::tree", "merge transitions")
+            .entered()
+            .in_scope(|| db.borrow_mut().merge_transitions(BundleRetention::Reverts));
         let output = BlockExecutionOutput { result, state: db.borrow_mut().take_bundle() };
 
         // Update the metrics for the number of accounts, storage slots and bytecodes updated