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

[ETCM-8382] feat: use paritytech polkadot-sdk #83

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

LGLO
Copy link
Contributor

@LGLO LGLO commented Sep 6, 2024

Description

Switches dependencies to Paritech polkadot-sdk.

We lose our fn starting_timestamp but there is a ticket to use fn timestamp that will be in the next polkadot stable release: https://input-output.atlassian.net/browse/ETCM-8382 with instructions where to use it.

Our major sync fork fix is not in polkadot-stable2407 as well, but seems that MC follower caching helped a lot here because the sync speed I've observed is 230-305 blocs per second, so it is approx x1500 speed - should be totally fine and with major sync fix it can get better.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages.
  • New tests are added if needed and existing tests are updated.
  • Relevant logging and metrics added
  • CI passes. See note on CI.
  • Any changes are noted in the changelog.md for affected crate
  • Self-reviewed the diff

Note on CI

If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges. Please contact IOG Partner Chains developers to do this
for you.

@LGLO LGLO changed the title feat: use paritytech polkadot-sdk [ETCM-8382] feat: use paritytech polkadot-sdk Sep 6, 2024
@LGLO LGLO force-pushed the ETCM-8344-use-paritytech-polkadot-sdk branch from a410925 to 0f12f3e Compare September 16, 2024 15:49
@LGLO LGLO merged commit 9201e54 into master Sep 18, 2024
8 checks passed
@LGLO LGLO deleted the ETCM-8344-use-paritytech-polkadot-sdk branch September 18, 2024 05:17
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.

4 participants