Skip to content

Commit

Permalink
[Cherry pick] cherry-pick execution_config changes into v1.6 (#8459 #…
Browse files Browse the repository at this point in the history
…9391) (#9390)

### Description

Commands (applied cleanly):
```
git revert f313b5b
git cherry-pick 66fe8cf
```


### Test Plan
<!-- Please provide us with clear details for verifying that your changes work. -->
  • Loading branch information
bchocho authored Jul 31, 2023
1 parent 409ffb3 commit 2ebf4eb
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
10 changes: 3 additions & 7 deletions consensus/src/epoch_manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,8 @@ use aptos_types::{
epoch_change::EpochChangeProof,
epoch_state::EpochState,
on_chain_config::{
ExecutionConfigV1, LeaderReputationType, OnChainConfigPayload, OnChainConsensusConfig,
OnChainExecutionConfig, ProposerElectionType, TransactionShufflerType, ValidatorSet,
LeaderReputationType, OnChainConfigPayload, OnChainConsensusConfig, OnChainExecutionConfig,
ProposerElectionType, ValidatorSet,
},
validator_verifier::ValidatorVerifier,
};
Expand Down Expand Up @@ -812,11 +812,7 @@ impl EpochManager {
match self.storage.start() {
LivenessStorageData::FullRecoveryData(initial_data) => {
let consensus_config = onchain_consensus_config.unwrap_or_default();
let execution_config = onchain_execution_config.unwrap_or(
OnChainExecutionConfig::V1(ExecutionConfigV1 {
transaction_shuffler_type: TransactionShufflerType::NoShuffling,
}),
);
let execution_config = onchain_execution_config.unwrap_or_default();
self.quorum_store_enabled = self.enable_quorum_store(&consensus_config);
self.recovery_mode = false;
self.start_round_manager(
Expand Down
6 changes: 3 additions & 3 deletions types/src/on_chain_config/execution_config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ impl OnChainExecutionConfig {
/// This is used when on-chain config is not initialized.
impl Default for OnChainExecutionConfig {
fn default() -> Self {
OnChainExecutionConfig::V1(ExecutionConfigV1::default())
OnChainExecutionConfig::V3(ExecutionConfigV3::default())
}
}

Expand Down Expand Up @@ -76,7 +76,7 @@ pub struct ExecutionConfigV1 {
impl Default for ExecutionConfigV1 {
fn default() -> Self {
Self {
transaction_shuffler_type: TransactionShufflerType::SenderAwareV1(32),
transaction_shuffler_type: TransactionShufflerType::NoShuffling,
}
}
}
Expand Down Expand Up @@ -108,7 +108,7 @@ impl Default for ExecutionConfigV3 {
Self {
transaction_shuffler_type: TransactionShufflerType::NoShuffling,
block_gas_limit: None,
transaction_deduper_type: TransactionDeduperType::NoDedup,
transaction_deduper_type: TransactionDeduperType::TxnHashAndAuthenticatorV1,
}
}
}
Expand Down

0 comments on commit 2ebf4eb

Please sign in to comment.