Remove type parameter A from RaftCore #234
Replies: 1 comment
-
too old. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
As of 0.7.1, RaftCore has
A: RaftApp
as the type parameter but we don't use this. The reason I added this is we can define an associated typeA::Snapshot
so the App implementer can give the type statically. However, with the abstraction snapshot tag this plan won't be done.Another drawback of this design aside being needless will become clear when we implement in-library multi-raft (#131). In there we could have multiple RaftCores that may have apps of different types. With the type parameter, we can't implement this.
Beta Was this translation helpful? Give feedback.
All reactions