-
Notifications
You must be signed in to change notification settings - Fork 524
AVM: Add an unlimited global storage mechanism - Boxes. #4001
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
Conversation
4b31e78 to
ac68b9d
Compare
4508dd7 to
7d2804f
Compare
michaeldiamant
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's bring the functionality into the feature branch for easier downstream consumption. 👍
|
This is really awesome, is there any ETA on when this is coming out? |
Keep your eyes on https://github.com/algorand/go-algorand/tree/feature/avm-box Probably August. |
This is the initial check-in to feature branch. I'll ask for reviews now, to shake out issues that I have not considered, but it will remain a WIP for a while longer, so final details are not important. We want to have a feature branch that other team members can work on.
Some things to think about/finish
do a fine job caching kv pairs, and there's not savings from skipping
msgpack serialization. But sqlite has a page cache, not a row cache,
so maybe it's worth it.
need to learn more about the rules that govern when the other lookup
routines are not taking lock.