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

Delegation compliance, simplified #8792

Merged
merged 3 commits into from
May 4, 2021

Conversation

psteckler
Copy link
Member

In delegation_compliance, check payments in the archive database directly, don't replay transactions. The updated app is able to check all the foundation delegations in about 11 seconds on my desktop. That's compared with 15 minutes for just one delegator with the existing code.

As input, provide a staking ledger, and the corresponding epoch number (0-based). Given the archive db, the app will find the longest canonical chain, and find the relevant transactions along the chain in the given epoch.

Tested with an archive db up to slot ~21000.

@psteckler psteckler added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label May 3, 2021
Copy link
Member

@deepthiskumar deepthiskumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

src/app/delegation_compliance/delegation_compliance.ml Outdated Show resolved Hide resolved
@mrmr1993 mrmr1993 merged commit c9257c8 into compatible May 4, 2021
@mrmr1993 mrmr1993 deleted the feature/simpler-delegation-compliance branch May 4, 2021 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants