Skip to content

Conversation

@HaoranYi
Copy link

@HaoranYi HaoranYi commented Apr 15, 2024

Problem

make combine_ancient_slots and shrink_ancient tests to work for both append-vec and hot-storages.

[NOTE] Current ancient storage algorithm is tired with shrink. It is based on the max_size of the storage to find candidate ancient slot. But this scheme won't work with HotStorage. We need to revisit the ancient storage algorithm for HotStorage...

Summary of Changes

  • refactor several accounts-db related test fn to support passing different type of account files
  • make combine_ancient_slots and shrink_ancient tests to work for both append-vec and hot-storages.

Fixes #

@mergify
Copy link

mergify bot commented Apr 15, 2024

⚠️ The sha of the head commit of this PR conflicts with #818. Mergify cannot evaluate rules on this PR. ⚠️

@HaoranYi HaoranYi force-pushed the accounts-db/refactor_accounts_db_test6 branch 4 times, most recently from 8f301d2 to bb006ea Compare April 19, 2024 15:20
@HaoranYi HaoranYi changed the title accounts db/refactor accounts db test6 accounts db/refactor accounts db test- make combine_ancient_slots and shrink_ancient tests to work for both append-vec and hot-storages Apr 19, 2024
@HaoranYi HaoranYi force-pushed the accounts-db/refactor_accounts_db_test6 branch 2 times, most recently from 2c78523 to c58455f Compare April 19, 2024 21:23
@HaoranYi HaoranYi force-pushed the accounts-db/refactor_accounts_db_test6 branch from c58455f to 525f399 Compare April 23, 2024 13:19
@github-actions
Copy link

This pull request is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Jan 12, 2026
@mergify mergify bot requested a review from a team January 12, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant