Skip to content

Commit

Permalink
[chore] disable the throughput aware consensus submission (#14536)
Browse files Browse the repository at this point in the history
## Description 

Disabling the throughput aware consensus submission for all environments

## Test Plan 

How did you test the new or updated feature?

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

### Type of Change (Check all that apply)

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration

### Release notes
  • Loading branch information
akichidis authored Oct 30, 2023
1 parent b5b8a59 commit 64fe2b6
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 7 deletions.
5 changes: 0 additions & 5 deletions crates/sui-protocol-config/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,6 @@ const MAX_PROTOCOL_VERSION: u64 = 30;
// Enable transaction effects v2 in testnet.
// Deprecate supported oauth providers from protocol config and rely on node config
// instead.
// Enable throughput aware submission for Devnet & Testnet
// In execution, has_public_transfer is recomputed when loading the object.
#[derive(Copy, Clone, Debug, Hash, Serialize, Deserialize, PartialEq, Eq, PartialOrd, Ord)]
pub struct ProtocolVersion(u64);
Expand Down Expand Up @@ -1624,10 +1623,6 @@ impl ProtocolConfig {
// whether the provider is supported based on node config.
cfg.feature_flags.zklogin_supported_providers = BTreeSet::default();

if chain != Chain::Mainnet {
cfg.feature_flags.throughput_aware_consensus_submission = true;
}

cfg.feature_flags.recompute_has_public_transfer_in_execution = true;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ feature_flags:
enable_effects_v2: true
narwhal_certificate_v2: true
verify_legacy_zklogin_address: true
throughput_aware_consensus_submission: true
recompute_has_public_transfer_in_execution: true
max_tx_size_bytes: 131072
max_input_objects: 2048
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ feature_flags:
enable_effects_v2: true
narwhal_certificate_v2: true
verify_legacy_zklogin_address: true
throughput_aware_consensus_submission: true
recompute_has_public_transfer_in_execution: true
max_tx_size_bytes: 131072
max_input_objects: 2048
Expand Down

0 comments on commit 64fe2b6

Please sign in to comment.