You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.
shawntabrizi opened this issue
Jun 3, 2021
· 2 comments
· Fixed by #9106
Labels
C3-mediumPR touches the given topic and has a medium impact on builders.Z6-mentorAn easy task where a mentor is available. Please indicate in the issue who the mentor could be.
When calling remove_all, a user should also be able to include Option<u32>, which describes the maximum number of keys that can be removed at once.
At the end of the operation, you can return AllRemoved(u32) or SomeReamining(u32) with the number of items that were actually removed at the end of the operation.
The text was updated successfully, but these errors were encountered:
shawntabrizi
added
Z6-mentor
An easy task where a mentor is available. Please indicate in the issue who the mentor could be.
C3-medium
PR touches the given topic and has a medium impact on builders.
labels
Jun 3, 2021
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
C3-mediumPR touches the given topic and has a medium impact on builders.Z6-mentorAn easy task where a mentor is available. Please indicate in the issue who the mentor could be.
Similar to the
storage_kill
api in child tries, theremove_all
api on any maps should also have an optional key limit to enforce weight restrictions.#8166
When calling
remove_all
, a user should also be able to includeOption<u32>
, which describes the maximum number of keys that can be removed at once.At the end of the operation, you can return
AllRemoved(u32)
orSomeReamining(u32)
with the number of items that were actually removed at the end of the operation.Should reuse and rename
KillChildStorageResult
:https://github.com/paritytech/substrate/blob/master/primitives/io/src/lib.rs#L89
The text was updated successfully, but these errors were encountered: