Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Generated votes cache size increase and upper bound #2432

Conversation

guilhermelawless
Copy link
Contributor

Memory footprint is insignificant compared to the cache for votes from other representatives.

@guilhermelawless guilhermelawless added performance Performance/resource utilization improvement networking labels Dec 9, 2019
@guilhermelawless guilhermelawless added this to the V21.0 milestone Dec 9, 2019
@guilhermelawless guilhermelawless self-assigned this Dec 9, 2019
@SergiySW
Copy link
Contributor

I guess it can use number of voting account in node as well to prevent overflow with current settings

@guilhermelawless guilhermelawless force-pushed the generated-votes-cache-size-increase branch from 85baced to 993fc00 Compare January 10, 2020 10:50
@guilhermelawless guilhermelawless changed the title Increase generated votes cache size to 64k from 4k Generated votes cache size increase and upper bound Jan 10, 2020
@guilhermelawless
Copy link
Contributor Author

@SergiySW updated, had to make a copy of representatives in wallet::foreach_representative to avoid a deadlock.

New test added and a weird clang-format change.

@guilhermelawless guilhermelawless force-pushed the generated-votes-cache-size-increase branch from ea6cac0 to 442cea0 Compare January 10, 2020 11:34
@guilhermelawless guilhermelawless merged commit eb2beac into nanocurrency:develop Jan 14, 2020
@guilhermelawless guilhermelawless deleted the generated-votes-cache-size-increase branch January 14, 2020 16:12
@guilhermelawless guilhermelawless removed the request for review from cryptocode February 7, 2020 09:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
networking performance Performance/resource utilization improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants