-
Notifications
You must be signed in to change notification settings - Fork 788
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
Stop Balances
pallet erroneously double incrementing and decrementing consumers
#1976
Conversation
Balances
pallet incorrectly double incrementing and decrementing consumersBalances
pallet erroneously double incrementing and decrementing consumers
First glance, this will be a race that you're alway behind, as more accounts can get their consumers incremented by mistake in the timespan while we wait for the migration to execute. |
We should first deploy the fix and then let the fix come in a later runtime upgrade. |
I agree, created a seperate issue for the fix #2037. So, this PR is ready for review and we should not unnecessarily delay merging it to reduce the total number of broken consumers. |
Balances
pallet erroneously double incrementing and decrementing consumersBalances
pallet erroneously double incrementing and decrementing consumers
…ng consumers (paritytech#1976) Closes paritytech#1970 Follow up issue to tackle, once the erroneous double incrementing/decrementing has stopped: paritytech#2037
What to do with chains that got fucked by this before the fix applied? |
Need to retroactively fix it #2037 |
Prepared this to see what needs fixing. Note that #1976 is not yet on Polkadot master, as it is on 1.4 - not 1.3. |
…ng consumers (paritytech#1976) Closes paritytech#1970 Follow up issue to tackle, once the erroneous double incrementing/decrementing has stopped: paritytech#2037
Closes #1970
Follow up issue to tackle, once the erroneous double incrementing/decrementing has stopped: #2037