Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Switch from Proposer to Slots #36

Closed
bkchr opened this issue Jan 15, 2020 · 4 comments · Fixed by #329
Closed

Switch from Proposer to Slots #36

bkchr opened this issue Jan 15, 2020 · 4 comments · Fixed by #329
Milestone

Comments

@bkchr
Copy link
Member

bkchr commented Jan 15, 2020

Instead of using Proposer as interface for building the blocks in the collator, we probably want to switch over to Slots as the interface. At the point when we want to support custom authoring, using Slots is easier as Babe and Aura already implemented using this trait.

We would need to change the trait in Substrate to return the Proposal, because we need this in the collator.

This was referenced Jun 15, 2020
@bkchr bkchr added this to the Velten milestone Jun 15, 2020
@JoshOrndorff
Copy link
Contributor

You mean switching to SlotWorker right? Or maybe SimpleSlotWorker?

Would this change eliminate the possibility of using PoW for collating? Is that a compromise we really want?

@bkchr
Copy link
Member Author

bkchr commented Jun 15, 2020

You mean switching to SlotWorker right? Or maybe SimpleSlotWorker?

SlotWorker.

Would this change eliminate the possibility of using PoW for collating? Is that a compromise we really want?

We don't really support PoW at the moment at all. So, yes.

@JoshOrndorff
Copy link
Contributor

I started looking at making the change to the SlotWorker trait, and I have a few questions about it.

  1. Proposal is generic over the Transaction type used in the backend. Should I make the SlotWorker trait also generic over this type?

  2. Currently the only implementation of the SlotWorker trait come from the blanket implementation over the SimpleSlotWorker trait.
    That means that I'll need to update the provided on_slot method for the SimpleSlotWorker trait. I'm stuck because I can't figure out how to return the proposal here. After it's data has already been moved eg here. It seems Proposal doesn't implement Clone.

  3. I see you mentioned on the Babe issue that it wouldn't happen soon. I'm curious if that is true for slots in general, or just for babe. (My main goal right now is Aura)

@bkchr
Copy link
Member Author

bkchr commented Feb 1, 2021

I'm already working on AURA, so you don't need to do this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants