-
Notifications
You must be signed in to change notification settings - Fork 102
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
Upgrade to v1.4.0-beta.1 & implement key-only Iteration #450
Conversation
c0cb00c
to
f0cc40f
Compare
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.
🚀
6b84068
to
7410295
Compare
223ba0f
to
b7d3f0f
Compare
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.
Great stuff 🎖️ This not only avoids copying the values around but never reads them from database.
Co-authored-by: Simon Warta <2603011+webmaster128@users.noreply.github.com>
Is this ready? Looks good as far as I can tell |
Yes, this is ready now (if we are fine with the 1.4.0-beta dependency) |
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.
Very nice
closes CosmWasm/cosmwasm#1498
Draft because CosmWasm/cosmwasm#1834 is not
mergedreleased yet (so I am using a dependency on the commit)