@ethDreamer pointed out the logging changes slightly - we exit early on an error, prior to logging how many requests we sent. The same changed in this PR: #5556

mark said he'd make a follow up PR for this though

For some reason, changing the Work enum in this PR caused a lint to fail. The changes here shouldn't affect the size of the enum, because we're just moving from Box to Pin<Box> in a variant. Looks like the enum was already big, but for some reason clippy didn't notice until the enum changed. This was the size before, there were two large variants:

print-type-size type: `Work<types::MainnetEthSpec>`: 192 bytes, alignment: 8 bytes
print-type-size     variant `GossipAggregate`: 192 bytes
print-type-size         field `.aggregate`: 160 bytes
print-type-size         field `.process_individual`: 16 bytes
print-type-size         field `.process_batch`: 16 bytes
print-type-size     variant `GossipAttestation`: 184 bytes
print-type-size         padding: 8 bytes
print-type-size         field `.attestation`: 144 bytes, alignment: 8 bytes
print-type-size         field `.process_individual`: 16 bytes
print-type-size         field `.process_batch`: 16 bytes

In this commit, I boxed the large unboxed fields in these variants 52777c9 , which reduced the size

print-type-size type: `Work<types::MainnetEthSpec>`: 64 bytes, alignment: 8 bytes

@Mergifyio queue

queue

✅ The pull request has been merged automatically