Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do periodic inbound cleaning for rooted slots (bp #8436) #8583

Merged
merged 1 commit into from
Mar 3, 2020

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Mar 3, 2020

This is an automated backport of pull request #8436 done by Mergify.io

Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR
  • @Mergifyio backports <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the simulator.

Finally, you can contact us on https://mergify.io/

* Do periodic inbound compaction for rooted slots

* Add comment

* nits

* Consider not_compacted_roots in cleanup_dead_slot

* Renames in AccountsIndex

* Rename to reflect expansion of removed accounts

* Fix a comment

* rename

* Parallelize clean over AccountsIndex

* Some niceties

* Reduce locks and real chunked parallelism

* Measure each step for sampling opportunities

* Just noticed par iter is maybe lazy

* Replace storage scan with optimized index scan

* Various clean-ups

* Clear uncleared_roots even if no updates

(cherry picked from commit d861033)
@mergify mergify bot added the automerge Merge this Pull Request automatically once CI passes label Mar 3, 2020
@solana-grimes solana-grimes merged commit f4385f7 into v1.0 Mar 3, 2020
@codecov
Copy link

codecov bot commented Mar 3, 2020

Codecov Report

Merging #8583 into v1.0 will increase coverage by <.1%.
The diff coverage is 98.8%.

@@           Coverage Diff           @@
##            v1.0   #8583     +/-   ##
=======================================
+ Coverage     80%   80.1%   +<.1%     
=======================================
  Files        256     256             
  Lines      55384   55518    +134     
=======================================
+ Hits       44355   44488    +133     
- Misses     11029   11030      +1

@mergify mergify bot deleted the mergify/bp/v1.0/pr-8436 branch March 3, 2020 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants