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 tinyint in migrate tool. #4842

Merged
merged 1 commit into from Feb 25, 2020
Merged

Conversation

arijitAD
Copy link

@arijitAD arijitAD commented Feb 24, 2020

Fixes #4674


This change is Reviewable

Copy link
Contributor

@martinmr martinmr left a comment

Choose a reason for hiding this comment

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

:lgtm: just make sure it works correctly. I don't know if there are automated tests for this tool but if there aren't a simple manual check should be enough.

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @mangalaman93 and @manishrjain)

Copy link
Author

@arijitAD arijitAD left a comment

Choose a reason for hiding this comment

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

I ran the migration tool manually to verify the schema and data.

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @mangalaman93 and @manishrjain)

@arijitAD arijitAD merged commit 87e1dcd into master Feb 25, 2020
@arijitAD arijitAD deleted the arijitAD/tinyint-migrate branch February 25, 2020 09:24
vardhanapoorv pushed a commit that referenced this pull request Mar 3, 2020
* Add support for tinyint in migrate tool.
arijitAD pushed a commit that referenced this pull request Apr 22, 2020
* Add support for tinyint in migrate tool.

(cherry picked from commit 87e1dcd)
danielmai pushed a commit that referenced this pull request Apr 22, 2020
* Add support for tinyint in migrate tool.

(cherry picked from commit 87e1dcd)
danielmai added a commit that referenced this pull request Apr 22, 2020
This reverts commit 79c68dc.

This was mistakenly cherry-picked to release/v1.2.2 instead of
release/v1.2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

MySQL tinyint type not handled my migrate tool
2 participants