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

Prioritization #3190

Merged
merged 5 commits into from
Apr 16, 2021
Merged

Prioritization #3190

merged 5 commits into from
Apr 16, 2021

Conversation

clemahieu
Copy link
Contributor

@clemahieu clemahieu commented Apr 16, 2021

This adds a class called prioritization which tracks accounts to be confirmed in a priority order determined by time_since_use * balance order where the time since use is per account.

The priority order is approximated by segmenting each account into a series of buckets determined by the number of leading 0 bits in the balance. Within each bucket, transactions are sorted by last-used timestamp of account where the numerically lowest last-used timestamp represents the account with the oldest modification date and thus, the highest priority.

…onfirmed in priority order determined by time_since_use * balance order where the time since use is per-account.

The priority order is approximated by segmented each account in to a series of buckets determined by the number of leading 0 bits in the balance. Within each bucket transactions are sorted by last-use timestamp of account where the numerically lowest last-use timestamp represents the account with the oldest modification date and thus, highest priority.
@clemahieu clemahieu merged commit 712f7a1 into develop Apr 16, 2021
@zhyatt zhyatt deleted the prioritization branch September 21, 2021 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants