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

Add support for Cloud Spanner data manipulation language #924

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

afarbos
Copy link

@afarbos afarbos commented May 5, 2023

Cloud Spanner databases provides multiple interfaces to interact with schema or data.
Today, (golang) migrate only support schema update (also called DDL). This PR will add support for data updates (DML).

This support is added with 0 changes required on the user side and full backward compatibility by relying on the parsing DML capabilities of the google cloud golang library.

Additionally, all lines added are tested.

This follow the issue #135

@afarbos afarbos force-pushed the spannerDML branch 2 times, most recently from f352ac3 to 0782496 Compare May 5, 2023 23:38
@afarbos afarbos changed the title Add support for spanner DML Add support for Cloud Spanner data manipulation language May 5, 2023
@coveralls
Copy link

coveralls commented May 6, 2023

Coverage Status

Coverage: 58.881%. Remained the same when pulling de05976 on afarbos:spannerDML into 84009cf on golang-migrate:master.

@afarbos afarbos force-pushed the spannerDML branch 3 times, most recently from 1461b30 to de05976 Compare May 6, 2023 02:07
@afarbos afarbos marked this pull request as ready for review May 6, 2023 02:46
@gorhamc
Copy link

gorhamc commented Aug 2, 2023

bump. can we get this PR reviewed? not sure who to tag @dhui maybe?

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.

None yet

3 participants