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

move epoch genesis to block processor #1720

Merged
merged 6 commits into from
Oct 1, 2024
Merged

Conversation

turbocrime
Copy link
Contributor

@turbocrime turbocrime commented Aug 20, 2024

idb is presently performing epoch genesis, instead of the block processor directing idb through the process.

block processor should contain all genesis special cases.

#1719

Copy link

changeset-bot bot commented Aug 20, 2024

🦋 Changeset detected

Latest commit: 5bb8f9d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@penumbra-zone/storage Minor
@penumbra-zone/query Minor
@penumbra-zone/services Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Base automatically changed from turbocrime/quick-sync to quick-sync August 20, 2024 18:19
Base automatically changed from quick-sync to main August 20, 2024 18:29
@TalDerei TalDerei requested a review from a team September 30, 2024 04:47
@TalDerei
Copy link
Contributor

TalDerei commented Sep 30, 2024

took over the draft PR and the opportunity to audit some of the database and block processor code for other instances of divergence where IDB may be performing tasks that are traditionally handled by the block processor. I think it's generally desirable to maintain a separation of concerns between different components.

packages/query/src/block-processor.ts Show resolved Hide resolved
packages/query/src/block-processor.ts Show resolved Hide resolved
packages/query/src/block-processor.ts Show resolved Hide resolved
packages/query/src/block-processor.ts Outdated Show resolved Hide resolved
packages/query/src/block-processor.ts Outdated Show resolved Hide resolved
packages/query/src/block-processor.ts Show resolved Hide resolved
packages/query/src/block-processor.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@grod220 grod220 left a comment

Choose a reason for hiding this comment

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

Mostly documentation review suggestions. After you address, feel free to merge.

@TalDerei TalDerei merged commit 487e26d into main Oct 1, 2024
6 checks passed
@TalDerei TalDerei deleted the turbocrime/epoch-genesis branch October 1, 2024 17:10
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.

3 participants