Skip to content

removes feature gated code enabling chained Merkle shreds#4787

Merged
behzadnouri merged 1 commit intoanza-xyz:masterfrom
behzadnouri:rm-enable-chained-merkle-shreds-feature
Feb 4, 2025
Merged

removes feature gated code enabling chained Merkle shreds#4787
behzadnouri merged 1 commit intoanza-xyz:masterfrom
behzadnouri:rm-enable-chained-merkle-shreds-feature

Conversation

@behzadnouri
Copy link
Copy Markdown

Problem

Chained Merkle shreds are enabled on all clusters.

$ solana -um feature status 7uZBkJXJ1HkuP6R3MJfZs7mLwymBcDbKdqbF51ZWLier
Feature                                      | Status                  | Activation Slot | Description
7uZBkJXJ1HkuP6R3MJfZs7mLwymBcDbKdqbF51ZWLier | active since epoch 710  | 306720000       | Enable chained Merkle shreds #34916

Summary of Changes

The commit removes feature gated code enabling chained Merkle shreds.


impl ShredFetchStage {
// updates packets received on a channel and sends them on another channel
#[allow(unused_variables, unused_assignments)]
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Need to keep feature checking machinery for #2149.
This will be removed in #2149.


// Returns true if the feature is effective for the shred slot.
#[must_use]
#[allow(dead_code)]
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Need to keep feature checking machinery for #2149.
This will be removed in #2149.

@behzadnouri behzadnouri merged commit f9bc2b7 into anza-xyz:master Feb 4, 2025
@behzadnouri behzadnouri deleted the rm-enable-chained-merkle-shreds-feature branch February 4, 2025 20:41
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.

3 participants