Revamp reaping rules #4371

gavofyork · 2019-12-11T21:34:55Z

Change DispatchQueue into a simple Vec item, which is fine as the entries are small and few (protected by governance). This makes is fast, cheap and easily enumerable.
Simplify some APIs due to the above change allowing us to search the queue.
Require a proposal preimage to not be in the queue for it to be reaped.
Allow the owner of a preimage to reap before the public at large.
Add a few tests.

…lse.

frame/democracy/src/lib.rs

Co-Authored-By: Shawn Tabrizi <shawntabrizi@gmail.com>

…o gav-less-hash-reaping-rules

shawntabrizi

LGTM

gavofyork added 2 commits December 12, 2019 04:59

Allow owner of a preimage to reap it a little while before everyone e…

5f00c25

…lse.

Revamp DispatchQueue to make reaping safer

1b7adfb

gavofyork requested a review from shawntabrizi December 11, 2019 21:34

Remove commented code

45041df

shawntabrizi reviewed Dec 11, 2019

View reviewed changes

frame/democracy/src/lib.rs Show resolved Hide resolved

shawntabrizi reviewed Dec 11, 2019

View reviewed changes

frame/democracy/src/lib.rs Outdated Show resolved Hide resolved

shawntabrizi reviewed Dec 11, 2019

View reviewed changes

frame/democracy/src/lib.rs Show resolved Hide resolved

gavofyork and others added 3 commits December 12, 2019 13:28

Update frame/democracy/src/lib.rs

03a0524

Co-Authored-By: Shawn Tabrizi <shawntabrizi@gmail.com>

Update docs

0dec926

Merge remote-tracking branch 'origin/gav-less-hash-reaping-rules' int…

957243e

…o gav-less-hash-reaping-rules

shawntabrizi approved these changes Dec 12, 2019

View reviewed changes

gavofyork merged commit d79e31e into master Dec 12, 2019

gavofyork deleted the gav-less-hash-reaping-rules branch December 12, 2019 10:24

Provide feedback