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

Bump avalanchego to v1.10.6-rc.4 #293

Merged
merged 2 commits into from
Aug 1, 2023
Merged

Bump avalanchego to v1.10.6-rc.4 #293

merged 2 commits into from
Aug 1, 2023

Conversation

aaronbuchwald
Copy link
Collaborator

@aaronbuchwald aaronbuchwald commented Aug 1, 2023

This PR bumps AvalancheGo dependency to use v1.10.6-rc.4 and updates the cache sizes used by the chain state cache to use the new sized LRU cache to limit the total memory allocated to the cache based off of bytes size (limited to a total of 20MB) as opposed to a specific number of blocks.

Since AvalancheGo imports Coreth, it will replace the version of AvalancheGo imported by Coreth with the actual current code when it's compiled. As a result, since AvalancheGo has switched to inputting sizes based off of the total memory taken up by these caches instead of the number of elements, we need to update the sizes for the AvalancheGo release.

Otherwise, the chain state cache will go from allocating 100/50/50 elements to the respective caches to allocating that number of bytes to each of those caches, which will end up being zero total blocks.

plugin/evm/vm.go Outdated Show resolved Hide resolved
@aaronbuchwald aaronbuchwald merged commit b360858 into master Aug 1, 2023
7 of 8 checks passed
@aaronbuchwald aaronbuchwald deleted the bump-avalanchego branch August 1, 2023 18:40
darioush pushed a commit that referenced this pull request Apr 25, 2024
* Bump avalanchego to v1.9.0

* go

* Update go.mod
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
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants