Skip to content

optimize LastSeenMessage#1834

Merged
shargon merged 9 commits intoneo-project:masterfrom
ZhangTao1596:fix-LastSeenMessage
Aug 12, 2020
Merged

optimize LastSeenMessage#1834
shargon merged 9 commits intoneo-project:masterfrom
ZhangTao1596:fix-LastSeenMessage

Conversation

@ZhangTao1596
Copy link

Apply #1802 on master.

  • Record LastSeenMessage precisely, especially after vote.

shargon
shargon previously approved these changes Aug 11, 2020
@erikzhang
Copy link
Member

In NEO3, the count of consensus nodes is fixed. So this is not needed, isn't it?

@ZhangTao1596
Copy link
Author

In NEO3, the count of consensus nodes is fixed. So this is not needed, isn't it?

What if validators changed. Still use previous record?

Co-authored-by: Shargon <shargon@gmail.com>
Copy link
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

erikzhang
erikzhang previously approved these changes Aug 12, 2020
@shargon shargon merged commit 6ded6df into neo-project:master Aug 12, 2020
@ZhangTao1596
Copy link
Author

👍

@ZhangTao1596 ZhangTao1596 deleted the fix-LastSeenMessage branch August 12, 2020 09:37
shargon added a commit that referenced this pull request Aug 13, 2020
* Fix 1834

* Fix UT

* Remove useless condition

Co-authored-by: Erik Zhang <erik@neo.org>
cloud8little pushed a commit to cloud8little/neo that referenced this pull request Jan 24, 2021
* optimize LastSeenMessage

* rm log

* use current height as default height

* Update src/neo/Consensus/ConsensusContext.cs

Co-authored-by: Shargon <shargon@gmail.com>

* clean

* Update ConsensusContext.cs

* Optimize

* Fix

Co-authored-by: Vitor Nazário Coelho <vncoelho@gmail.com>
Co-authored-by: Shargon <shargon@gmail.com>
Co-authored-by: erikzhang <erik@neo.org>
cloud8little pushed a commit to cloud8little/neo that referenced this pull request Jan 24, 2021
* Fix 1834

* Fix UT

* Remove useless condition

Co-authored-by: Erik Zhang <erik@neo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants