-
Notifications
You must be signed in to change notification settings - Fork 56
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
MySQL foreign key action handling at the backend with warning generation #854
MySQL foreign key action handling at the backend with warning generation #854
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, very well written code with great test coverage across MySQL dump and PG dialect as well.
I also see some new tests added for methods that you only modified. Thanks for going above and beyond.
LGTM with minor comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Great work on adding good test coverage and quality code with comments.
Tasks Handled in this PR:
a. Inferring source actions
b. Conversion of action from source schema to spanner schema
c. DDL generation
Unit tests for existing functions were extended and new unit tests were added for new functions.