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

slot-based-collator: Refactor some internals #6935

Merged
merged 3 commits into from
Dec 18, 2024

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Dec 17, 2024

Move RelayChainDataFetcher to its own module and rename it to RelayChainDataCache. Also move the core_selector function to the slot_based module.

Related issue: #6495

Move `RelayChainDataFetcher` to its own module and rename it to `RelayChainDataCache`. Also move the `core_selector` function to the `slot_based` module.
These changes are required for the 500ms blocks.
@bkchr bkchr added the R0-silent Changes should not be mentioned in any release notes label Dec 17, 2024
@bkchr bkchr requested review from skunert and a team December 17, 2024 21:19
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/12381926463
Failed job name: test-linux-stable

@bkchr bkchr enabled auto-merge December 18, 2024 13:51
@bkchr bkchr added this pull request to the merge queue Dec 18, 2024
Merged via the queue into master with commit fd0fb76 Dec 18, 2024
198 of 201 checks passed
@bkchr bkchr deleted the bkchr-slot-based-relay-chain-data-cache branch December 18, 2024 15:00
dudo50 pushed a commit to paraspell-research/polkadot-sdk that referenced this pull request Jan 4, 2025
Move `RelayChainDataFetcher` to its own module and rename it to
`RelayChainDataCache`. Also move the `core_selector` function to the
`slot_based` module.

Related issue: paritytech#6495
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants