Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Additional ledger stats #2515

Merged

Conversation

guilhermelawless
Copy link
Contributor

Stats have consistently been helpful and leading to many optimizations, these are intended to help grasp the efficacy of the duplicate filter and bottlenecks in single-account processing.

Adds detail stats when ledger processing from the block processor, for old, gap_previous and gap_source process codes.

@guilhermelawless guilhermelawless added the documentation This item indicates the need for or supplies updated or expanded documentation label Jan 27, 2020
@guilhermelawless guilhermelawless added this to the V21.0 milestone Jan 27, 2020
@guilhermelawless guilhermelawless self-assigned this Jan 27, 2020
@guilhermelawless guilhermelawless merged commit 89a19de into nanocurrency:develop Jan 27, 2020
@guilhermelawless guilhermelawless deleted the stats/ledger-extra branch January 27, 2020 18:56
@zhyatt zhyatt added the debug Updates assisting with debugging and development efforts label Apr 21, 2020
@zhyatt
Copy link
Collaborator

zhyatt commented Apr 21, 2020

@guilhermelawless Have these details been documented already?

guilhermelawless added a commit to nanocurrency/nano-docs that referenced this pull request Apr 22, 2020
@guilhermelawless
Copy link
Contributor Author

Thanks @zhyatt , just linked a PR above

guilhermelawless added a commit to nanocurrency/nano-docs that referenced this pull request Apr 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debug Updates assisting with debugging and development efforts documentation This item indicates the need for or supplies updated or expanded documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants