From 49d1b50ca5ee11f679c19d913e74470c68b80cf1 Mon Sep 17 00:00:00 2001 From: Alexey Shekhirin <5773434+shekhirin@users.noreply.github.com> Date: Mon, 6 Jan 2025 11:06:10 +0000 Subject: [PATCH] chore(trie): parallel storage proof should not error on closed recv (#13448) --- crates/trie/parallel/src/proof.rs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/crates/trie/parallel/src/proof.rs b/crates/trie/parallel/src/proof.rs index ef7e34b19..ce0c185e1 100644 --- a/crates/trie/parallel/src/proof.rs +++ b/crates/trie/parallel/src/proof.rs @@ -25,7 +25,7 @@ use reth_trie::{ use reth_trie_common::proof::ProofRetainer; use reth_trie_db::{DatabaseHashedCursorFactory, DatabaseTrieCursorFactory}; use std::{sync::Arc, time::Instant}; -use tracing::{debug, error, trace}; +use tracing::{debug, trace}; #[cfg(feature = "metrics")] use crate::metrics::ParallelStateRootMetrics; @@ -196,8 +196,11 @@ where proof_result })(); + // We can have the receiver dropped before we send, because we still calculate + // storage proofs for deleted accounts, but do not actually walk over them in + // `account_node_iter` below. if let Err(e) = tx.send(result) { - error!( + debug!( target: "trie::parallel", ?hashed_address, error = ?e,