Use same max_age regardless of leader/not-leader (#9423)

automerge
This commit is contained in:
Ryo Onodera 2020-04-14 16:34:41 +09:00 committed by GitHub
parent 2d0224b64e
commit c77ed82caa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 6 deletions

View File

@ -20,7 +20,7 @@ use solana_runtime::{
transaction_batch::TransactionBatch, transaction_batch::TransactionBatch,
}; };
use solana_sdk::{ use solana_sdk::{
clock::{Slot, MAX_RECENT_BLOCKHASHES}, clock::{Slot, MAX_PROCESSING_AGE},
genesis_config::GenesisConfig, genesis_config::GenesisConfig,
hash::Hash, hash::Hash,
pubkey::Pubkey, pubkey::Pubkey,
@ -70,7 +70,7 @@ fn execute_batch(
balances, balances,
) = batch.bank().load_execute_and_commit_transactions( ) = batch.bank().load_execute_and_commit_transactions(
batch, batch,
MAX_RECENT_BLOCKHASHES, MAX_PROCESSING_AGE,
transaction_status_sender.is_some(), transaction_status_sender.is_some(),
); );

View File

@ -32,7 +32,10 @@ use solana_metrics::{
}; };
use solana_sdk::{ use solana_sdk::{
account::Account, account::Account,
clock::{get_segment_from_slot, Epoch, Slot, UnixTimestamp, MAX_RECENT_BLOCKHASHES}, clock::{
get_segment_from_slot, Epoch, Slot, UnixTimestamp, MAX_PROCESSING_AGE,
MAX_RECENT_BLOCKHASHES,
},
epoch_schedule::EpochSchedule, epoch_schedule::EpochSchedule,
fee_calculator::{FeeCalculator, FeeRateGovernor}, fee_calculator::{FeeCalculator, FeeRateGovernor},
genesis_config::GenesisConfig, genesis_config::GenesisConfig,
@ -1671,7 +1674,7 @@ impl Bank {
#[must_use] #[must_use]
pub fn process_transactions(&self, txs: &[Transaction]) -> Vec<Result<()>> { pub fn process_transactions(&self, txs: &[Transaction]) -> Vec<Result<()>> {
let batch = self.prepare_batch(txs, None); let batch = self.prepare_batch(txs, None);
self.load_execute_and_commit_transactions(&batch, MAX_RECENT_BLOCKHASHES, false) self.load_execute_and_commit_transactions(&batch, MAX_PROCESSING_AGE, false)
.0 .0
.fee_collection_results .fee_collection_results
} }
@ -3859,7 +3862,7 @@ mod tests {
let lock_result = bank.prepare_batch(&pay_alice, None); let lock_result = bank.prepare_batch(&pay_alice, None);
let results_alice = bank let results_alice = bank
.load_execute_and_commit_transactions(&lock_result, MAX_RECENT_BLOCKHASHES, false) .load_execute_and_commit_transactions(&lock_result, MAX_PROCESSING_AGE, false)
.0 .0
.fee_collection_results; .fee_collection_results;
assert_eq!(results_alice[0], Ok(())); assert_eq!(results_alice[0], Ok(()));
@ -5640,7 +5643,7 @@ mod tests {
let lock_result = bank0.prepare_batch(&txs, None); let lock_result = bank0.prepare_batch(&txs, None);
let (transaction_results, transaction_balances_set) = let (transaction_results, transaction_balances_set) =
bank0.load_execute_and_commit_transactions(&lock_result, MAX_RECENT_BLOCKHASHES, true); bank0.load_execute_and_commit_transactions(&lock_result, MAX_PROCESSING_AGE, true);
assert_eq!(transaction_balances_set.pre_balances.len(), 3); assert_eq!(transaction_balances_set.pre_balances.len(), 3);
assert_eq!(transaction_balances_set.post_balances.len(), 3); assert_eq!(transaction_balances_set.post_balances.len(), 3);