Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Minor cleanup for accounts-cluster-bench #16248

Merged
merged 1 commit into from
Mar 31, 2021

Conversation

CriesofCarrots
Copy link
Contributor

Problem

Attempting to close accounts in accounts-cluster-bench may cause some confusing errors.

Summary of Changes

  • Use const for fee calculation to prevent incrementing max_created atomic
  • Expand arg help text to explain other potential error

@CriesofCarrots CriesofCarrots added v1.6 automerge Merge this Pull Request automatically once CI passes labels Mar 31, 2021
@codecov
Copy link

codecov bot commented Mar 31, 2021

Codecov Report

Merging #16248 (7e3c49c) into master (aac18d7) will decrease coverage by 0.0%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #16248     +/-   ##
=========================================
- Coverage    80.0%    80.0%   -0.1%     
=========================================
  Files         410      410             
  Lines      109609   109609             
=========================================
- Hits        87785    87772     -13     
- Misses      21824    21837     +13     

@mergify mergify bot merged commit 67b7479 into solana-labs:master Mar 31, 2021
mergify bot pushed a commit that referenced this pull request Mar 31, 2021
(cherry picked from commit 67b7479)
mergify bot added a commit that referenced this pull request Mar 31, 2021
(cherry picked from commit 67b7479)

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
@CriesofCarrots CriesofCarrots deleted the cleanup-many-accounts branch April 6, 2021 16:41
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant