Quota pallet benchmark
Merge request reports
Activity
assigned to @bgallois
added 12 commits
-
869af9bd...b793b455 - 11 commits from branch
nodes/rust:master
- 4b5ca95a - add pallet-quota benchmarks
-
869af9bd...b793b455 - 11 commits from branch
added 2 commits
added RN-runtime label
changed milestone to %runtime-700
Added benchmarks for the quota pallet and fixed an error in the pallet identity benchmarks from merge request !183 (merged).
requested review from @HugoTrentesaux
In quota
tests.rs
, there is this line:// TODO implement a mock weight to test if refund queue processing actually stops when reached limit
I think it would be good to add a test to make sure that
on_idle
stops before consuming more weight than available.In quota
lib.rs
, there is this line:// opti: benchmark process_refund_queue overhead and substract this from weight limit
I think
process_refund_queue
overhead is quite small, but to be clean, we have to make sure that it does not lead to exceed theremaining_weight
given toon_idle
.These could be added in this MR.
And the pallet identity benchmark fix could go to !188 (merged), it would be a good example of its use case! (and I think the
IdentitiesNames
was changed in !182 (merged), not !183 (merged) )added 4 commits
-
8857e186...2ddae276 - 2 commits from branch
nodes/rust:master
- c9df48e7 - add pallet-quota benchmarks
- da8e289c - benchmark process_refund_queue overhead
-
8857e186...2ddae276 - 2 commits from branch