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

Fix: update balance table #849

Merged
merged 2 commits into from
Nov 16, 2020
Merged

Fix: update balance table #849

merged 2 commits into from
Nov 16, 2020

Conversation

austinabell
Copy link
Contributor

@austinabell austinabell commented Nov 13, 2020

Summary of changes
Changes introduced in this pull request:

  • Balance table had changed in actors, but wasn't noticed because the only difference was that items in the table are now removed when the balance goes back to 0 (somewhat strange as it would incur a large gas cost if not encountered in a cron, but I guess this is just to avoid state bloat)

With #847, syncs past 3388 (still syncing atm)

Reference issue to close (if applicable)

Closes

Other information and links

@austinabell austinabell mentioned this pull request Nov 16, 2020
@austinabell austinabell merged commit 396052c into main Nov 16, 2020
@austinabell austinabell deleted the austin/fixbalancetable branch November 16, 2020 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Interop Interop with Lotus/specs-actors and testing Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants