sui Use execution driver instead of sync_authority_source_to

Use execution driver instead of sync_authority_source_to_destination for gossip

Open mystenmark opened this issue 3 years ago • 1 comments

This improves parallelism, and removes the use of the (slightly) incorrect sync_authority_source_to_destination

Jul 26 '22 18:07 mystenmark

What is slightly incorrect about sync_authority_source_to_destination?

It trusts the effects (specifically the parent certs) given to it by the source authority, which may in fact be byzantine.

The correct way to get effects is at https://github.com/MystenLabs/sui/blob/main/crates/sui-core/src/authority_aggregator.rs#L1842

Jul 27 '22 00:07 mystenmark