Wrap BlobTransactionSideCar in Arcs in txpool

[mattsse]

Describe the feature

currently, we're cloning the sidecar in various places, which is pretty expensive, we also never modify these so we really should pass that around as an Arc.

this is pretty common and we should actually add a helper type like SharedBlobTransactionSidecar(Arc<BlobTransactionSidecar>) to alloy directly.

TODO

• introduce helper type in alloy
• transition to that type

since this type is very simple, this can also added to reth so we don't need to wait until it lands in alloy.

e.g:

reth/crates/transaction-pool/src/traits.rs

Lines 405 to 408 in ac85fc0

	fn get_all_blobs(
	&self,
	tx_hashes: Vec<TxHash>,
	) -> Result<Vec<(TxHash, BlobTransactionSidecar)>, BlobStoreError>;

Additional context

No response

[kdonthi]

I would like to take this

[mattsse]

actually we can just use Arc<BlobTransactionSidecar> everywhere first

like we already do here:

reth/crates/transaction-pool/src/traits.rs

Line 579 in 0ce1dd6

pub sidecar: Arc<BlobTransactionSidecar>,

we still want the helper type on alloy though

[YashBit]

@mattsse I am interested in some good first issues as well. But they always seem to be taken up quickly. Any other recommendations for issues which don't have the label, but might be relatively easier to get started with?