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

force the reward pulser to completion sooner #2521

Merged
merged 1 commit into from
Oct 17, 2021

Conversation

JaredCorduan
Copy link
Contributor

Some tools, such as db-sync and the stake credential history tool, rely on being able to inspect the ledger state for the reward update. If the pulser does not finish before the end of the epoch (which has happened on the Shelley-QA network, see
IntersectMBO/cardano-db-sync#882) then the pulser is forced to completion at the moment it is applied, as is therefore invisible to downstream tools.

To solve this problem, we force completion of the reward pulser by (2k/f)-many slots before the end of the epoch (one day on mainnet).

Some tools, such ad db-sync and the stake credential history tool rely
on being able to inspect the ledger state for the reward update. If the
pulser does not finish before the end of the epoch (which has happened
on the Shelley-QA network, see
IntersectMBO/cardano-db-sync#882) then the
pulser is forced to completion at the moment it is applied, as is
therefore invisible to downstream tools.

To solve this problem, we force completion of the reward pulser by
(2k/f)-many slots before the end of the epoch (one day on mainnet).
@JaredCorduan JaredCorduan force-pushed the jc/force-reward-pulser-before-end-of-epoch branch from 3dd8080 to 4e43452 Compare October 14, 2021 21:27
@erikd erikd merged commit 5477ee6 into master Oct 17, 2021
@iohk-bors iohk-bors bot deleted the jc/force-reward-pulser-before-end-of-epoch branch October 17, 2021 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants