-
Notifications
You must be signed in to change notification settings - Fork 217
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
HWY-249: Add the timestamp to CandidateBlock
.
#879
Conversation
Validity of a proto block depends on the timestamp. If there are two different candidate blocks (belonging to different units) with the same proto block, we must not apply the validity result to both.
@@ -192,20 +192,25 @@ impl<I> Era<I> { | |||
pub(crate) fn resolve_validity( | |||
&mut self, | |||
proto_block: &ProtoBlock, | |||
timestamp: Timestamp, |
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.
This is a timestamp of the unit that proposed the ProtoBlock
?
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.
Yes
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.
…and it will be the timestamp of the FinalizedBlock
and Block
itself.
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.
Can you add a comment or improve the name? In order to figure out what the timestamp
is here one has to go up a couple of levels and then down into another branch where it was created.
bors r+ |
Build succeeded: |
Validity of a proto block depends on the timestamp. If there are two different candidate blocks (belonging to different units) with the same proto block, we must not apply the validity result to both.
https://casperlabs.atlassian.net/browse/HWY-249