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

[consensus] Enable checkpoint batching and leader schedule change for mysticeti mainnet #18564

Merged
merged 3 commits into from
Jul 11, 2024

Conversation

arun-koshy
Copy link
Contributor

@arun-koshy arun-koshy commented Jul 8, 2024

Description

Enable features in protocol config (v52) for Mysticeti mainnet launch

  • Checkpoint batching
  • Leader Scoring & Schedule Change

Removed v53 and moved feature changes to v52


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • [ X ] Protocol: v52
    -- Enable checkpoint batching in mainnet.
    -- Enable Leader Scoring & Schedule Change for Mysticeti consensus on mainnet.

@arun-koshy arun-koshy requested a review from mystenmark as a code owner July 8, 2024 21:51
Copy link

vercel bot commented Jul 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2024 9:08pm
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Jul 11, 2024 9:08pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Jul 11, 2024 9:08pm
sui-typescript-docs ⬜️ Ignored (Inspect) Visit Preview Jul 11, 2024 9:08pm


// Enable checkpoint batching on mainnet.
cfg.checkpoint_summary_version_specific_data = Some(1);
cfg.min_checkpoint_interval_ms = Some(200);
}
53 => {
Copy link
Contributor

Choose a reason for hiding this comment

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

since 52 was reverted from testnet branch, do we need to introduce 53 right now? Or should we collapse them?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah good point, 52 & 53 would get released together anyways so I guess there is no harm in collapsing them into 52.

@arun-koshy arun-koshy merged commit 67874f2 into main Jul 11, 2024
48 checks passed
@arun-koshy arun-koshy deleted the ak/leader-schedule-mainnet branch July 11, 2024 21:27
tx-tomcat pushed a commit to tx-tomcat/sui-network that referenced this pull request Jul 29, 2024
… mysticeti mainnet (MystenLabs#18564)

## Description 

Enable features in protocol config (v52) for Mysticeti mainnet launch
- Checkpoint batching
- Leader Scoring & Schedule Change

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ X ] Protocol: v52 
-- Enable checkpoint batching in mainnet.
-- Enable Leader Scoring & Schedule Change for Mysticeti consensus on
mainnet.
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.

2 participants