casper-network / casper-network/casper-node

Memoize deploy's approvals hash if beneficial

Open
#4,062 1 comment 0 reactions 0 assignees View on GitHub
performance stretch goal tech-debt
Dominant language
Rust
Stars
399
Forks
224
Avg merge
15h 44m
Merged PRs (30d)
2

Description

We should measure how often Deploy::compute_approvals_hash is called per deploy, and if it is more than once or twice, then we should memoize this result in the same style as for Deploy::is_valid.

Measure first and see if it is worth the effort.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.