Skip to content
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

Release specs v1.9.0 #57

Merged
merged 4 commits into from
Mar 31, 2025
Merged

Release specs v1.9.0 #57

merged 4 commits into from
Mar 31, 2025

Conversation

sstanculeanu
Copy link
Contributor

No description provided.

@sstanculeanu sstanculeanu self-assigned this Mar 21, 2025
@sasurobert sasurobert self-requested a review March 21, 2025 13:21
sasurobert
sasurobert previously approved these changes Mar 21, 2025
mariusmihaic
mariusmihaic previously approved these changes Mar 21, 2025

Previously, only the leader handled the aggregated signature and set it on the initially proposed block, but now it can be created and propagated as a separate equivalent proof by every eligible node in consensus. These proofs are equivalent since they represent a majority from the eligible nodes in consensus.

The proof will be propagated in the network only if it was created directly. Otherwise, it will not be further propagated when received from other nodes in the network, to avoid flooding.

Choose a reason for hiding this comment

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

An equivalent consensus proof will be propagated only once by every node, either if it was created by the node or if it was the first proof received and validated by the node.

- [#4819](https://github.com/multiversx/mx-chain-go/pull/4819) - Fixes for nodes coordinator with chain parameters
- [#4927](https://github.com/multiversx/mx-chain-go/pull/4927) - Chain parameters notifier

## 2. Equivalent proofs [#5786](https://github.com/multiversx/mx-chain-go/pull/5786)

Choose a reason for hiding this comment

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

Can we split this into 2 parts?

  1. Fixed ordering in consensus

  2. Equivalent proofs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

pushed


This document explains the contents of the release codenamed Andromeda.

Although it can be considered one big feature, this document is split into 2 features, containing detailed insights of the feature along with the external impact and the relevant pull requests list.

Choose a reason for hiding this comment

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

into 3 features

@sstanculeanu sstanculeanu dismissed stale reviews from mariusmihaic and sasurobert via 5d8a3a3 March 24, 2025 13:53

## 2. Fixed ordering in consensus [#6436](https://github.com/multiversx/mx-chain-go/pull/6436)

Each eligible node is now required to sign, as the order of signing was moved to a fixed 400 value.
Copy link

@AdoAdoAdo AdoAdoAdo Mar 24, 2025

Choose a reason for hiding this comment

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

each eligible node is required to sign and the order of signatures aggregation was fixed to the order of the eligible nodes decided at epoch start.


An equivalent consensus proof will be propagated only once by every node, either if it was created by the node or if it was the first proof received and validated by the node.

The proof for the current block will be included in the next block structure. A particular block is considered final if there is an equivalent proof for it.

Choose a reason for hiding this comment

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

if there exists an equivalent ...

@sstanculeanu sstanculeanu merged commit 3168798 into main Mar 31, 2025
@sstanculeanu sstanculeanu deleted the release_protocol_v1.9.0 branch March 31, 2025 14:40
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.

5 participants