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

Survival waves #2308

Closed
wants to merge 202 commits into from
Closed

Survival waves #2308

wants to merge 202 commits into from

Conversation

tkstanczak
Copy link
Member

@tkstanczak tkstanczak commented Sep 13, 2020

For review:

  • MemCountingCache (check if tested, check if needed)
  • NUNitLogManagerand NUnitLogger
  • TrieNodeChae, TrieStore, TrieNode, PatriciaTriee changes for pruning
  • IPruningStrategy, ISnapshotStrategy
  • important resettable fix (adding missing comparer)
  • addition of logging to PatriciaTrie and nodes

Remaining items:

  • cleaning the TrieStore / TrieCache relationship
  • cleaning the PruningStrategy and SnapshotStrategy rules
  • cleaning the propagation of TrieStore
  • fixing tests
  • adding tests for the two replace-storage cases
  • adding test cases for the short RLP
  • reacting to memory growth in the cache
  • support for reorgs (already there, needs connecting and testing)

@tkstanczak tkstanczak added this to In progress in October 2020 Sep 29, 2020
@tkstanczak tkstanczak closed this Oct 17, 2020
October 2020 automation moved this from In progress to Done Oct 17, 2020
@rubo rubo deleted the survivalWaves branch December 20, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant