Not allow transactions request zero CU to update prioritization fee cache (#30507)

* filter out transaction requests zero cu from updating prioritization fee cache

* updated comment
This commit is contained in:
Tao Zhu 2023-02-24 23:50:02 -06:00 committed by GitHub
parent 9ec5e9b866
commit cbcd3ef27e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 1 deletions

View File

@ -219,6 +219,13 @@ impl PrioritizationFeeCache {
if priority_details.is_none() || account_locks.is_err() { if priority_details.is_none() || account_locks.is_err() {
continue; continue;
} }
let priority_details = priority_details.unwrap();
// filter out any transaction that requests zero compute_unit_limit
// since its priority fee amount is not instructive
if priority_details.compute_unit_limit == 0 {
continue;
}
let writable_accounts = Arc::new( let writable_accounts = Arc::new(
account_locks account_locks
@ -232,7 +239,7 @@ impl PrioritizationFeeCache {
self.sender self.sender
.send(CacheServiceUpdate::TransactionUpdate { .send(CacheServiceUpdate::TransactionUpdate {
slot: bank.slot(), slot: bank.slot(),
transaction_fee: priority_details.unwrap().priority, transaction_fee: priority_details.priority,
writable_accounts, writable_accounts,
}) })
.unwrap_or_else(|err| { .unwrap_or_else(|err| {