narwhal icon indicating copy to clipboard operation
narwhal copied to clipboard

feat: let the Primary's core use the Consensus recovery

Open huitseeker opened this issue 3 years ago • 1 comments

This is a superset of #814:

  • this front-loads the recovery of the consensus, creating its state before actually spawning the task,
  • then modifies the recovery of the primary to use the result of consensus recovery,

Note that we do not provide a recovery to the primary in case we are running with an external consensus. In this case, we simply follow the logic outlined in #814.

Opening for review for comments, this requires more testing.

Part of #776

huitseeker avatar Aug 18 '22 18:08 huitseeker

The simplest way to bring this to a close (w/o spurious code changes) is that this will need to be refactored on top of (and is blocked by) MystenLabs/narwhal#825.

huitseeker avatar Aug 25 '22 16:08 huitseeker

Closing the PR as newer work has been done under Narwhal's new home https://github.com/MystenLabs/sui

akichidis avatar Oct 17 '22 13:10 akichidis