Skip to content

Scylla migration support #2085

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

Open
wants to merge 6 commits into
base: development
Choose a base branch
from

Conversation

om7057
Copy link

@om7057 om7057 commented Jul 21, 2025

Description:

  • This PR introduces support for ScyllaDB migration testing within the migration package.
  • Addresses setup and initialization issues related to MockScyllaDB, ensuring stable test execution.

Breaking Changes (if applicable):

  • None.

Additional Information:

  • Relies on gomock for mocking the ScyllaDB interface.
  • Main file : scylla_db.go
  • Related test file: scylla_db_test.go.

Checklist:

  • I have formatted my code using goimport and golangci-lint.
  • All new code is covered by unit tests.
  • This PR does not decrease the overall code coverage.
  • I have reviewed the code comments and documentation for clarity.

Thank you for your contribution!

@om7057
Copy link
Author

om7057 commented Jul 21, 2025

@coolwednesday and @Umang01-hash I've added the respective changes for ScyllaDB migration support, please review and do let me know further updates.

@om7057
Copy link
Author

om7057 commented Jul 23, 2025

Hi @Umang01-hash and @coolwednesday I've fixed some of the merge conflicts, kindly go through once.

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