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

feat(cat-gateway): Add autogenerated cassandra schema version and sync status tables. #889

Merged
merged 129 commits into from
Sep 28, 2024

Conversation

stevenj
Copy link
Collaborator

@stevenj stevenj commented Sep 26, 2024

Description

  • Autogenerate the schema version for Cassandra.
  • Check in CI if it has changed unintentionally.
  • Add the Sync Status tables to the index db.
  • Clean up how blake2b hashes are generated by moving everything to a utility module.

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

…ainnet, adjust `justfile` to properly select preprod and also refresh git dependencies.
Copy link
Contributor

github-actions bot commented Sep 26, 2024

Test Report | ${\color{lightgreen}Pass: 272/272}$ | ${\color{red}Fail: 0/272}$ |

@stevenj stevenj self-assigned this Sep 26, 2024
@stevenj stevenj added the review me PR is ready for review label Sep 26, 2024
Mr-Leshiy
Mr-Leshiy previously approved these changes Sep 27, 2024
Copy link
Contributor

@Mr-Leshiy Mr-Leshiy left a comment

Choose a reason for hiding this comment

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

Despite one small comment LGTM

@stevenj
Copy link
Collaborator Author

stevenj commented Sep 28, 2024

Docs are broken upstream.
I have replaced the necessary version in both catalyst-forge and catalyst-ci, but neither seem to be taking effect.
Force merging, as this CI breakage is outside of our control, and everything else seems to work.

@stevenj stevenj merged commit ff7a0d7 into main Sep 28, 2024
31 of 32 checks passed
@stevenj stevenj deleted the feat/chain-sync-v2 branch September 28, 2024 10:24
@stevenj stevenj restored the feat/chain-sync-v2 branch September 28, 2024 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review me PR is ready for review
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants