-
Notifications
You must be signed in to change notification settings - Fork 33
ObjectDiffusion v2 (just inbound side changes) for efficient vote retrieval #1698
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: peras-staging
Are you sure you want to change the base?
ObjectDiffusion v2 (just inbound side changes) for efficient vote retrieval #1698
Conversation
96b7016 to
8bd6007
Compare
d28fe49 to
f794d6f
Compare
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.md
Outdated
Show resolved
Hide resolved
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.md
Outdated
Show resolved
Hide resolved
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.md
Outdated
Show resolved
Hide resolved
a459164 to
7fa719c
Compare
987f215 to
56c9c48
Compare
a97ac93 to
eef954f
Compare
94eb9ce to
c6db8de
Compare
…iniProtocol.ObjectDiffusion.Inbound.V1`
c6db8de to
1a1c81d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some initial comments. I still need to take a look in depth at:
-
ObjectDiffusion/Inbound/V2/Decision.hs -
ObjectDiffusion/Inbound/V2/Registry.hs -
ObjectDiffusion/Inbound/V2/State.hs - Benchmarking commit
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.hs
Outdated
Show resolved
Hide resolved
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.hs
Outdated
Show resolved
Hide resolved
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.hs
Outdated
Show resolved
Hide resolved
...ensus/src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2.hs
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Types.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Types.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Types.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Types.hs
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Types.hs
Outdated
Show resolved
Hide resolved
agustinmista
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another round of comments.
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Decision.hs
Outdated
Show resolved
Hide resolved
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Decision.hs
Outdated
Show resolved
Hide resolved
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Decision.hs
Outdated
Show resolved
Hide resolved
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Decision.hs
Outdated
Show resolved
Hide resolved
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Decision.hs
Outdated
Show resolved
Hide resolved
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Registry.hs
Outdated
Show resolved
Hide resolved
.../ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Registry.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/State.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/State.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/State.hs
Outdated
Show resolved
Hide resolved
...src/ouroboros-consensus/Ouroboros/Consensus/MiniProtocol/ObjectDiffusion/Inbound/V2/Types.hs
Outdated
Show resolved
Hide resolved
|
FTR, I'm a bit unsure about introducing a proper data type In the end, instead of a single But I don't see a nicer alternative atm than what we already had before or what we have now... I initially used a plain tuple instead of the new |
b95c960 to
95c23db
Compare
|
I pushed a few changes:
|
95c23db to
7219242
Compare
Co-authored-by: nbacquey nbacquey@users.noreply.github.com
Description
Please include a meaningful description of the PR and link the relevant issues
this PR might resolve.
Also note that: