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

Backward compatible schema discovery for MySQL 5.6 #86

Merged
merged 5 commits into from
Nov 16, 2022

Conversation

billy1624
Copy link
Member

@billy1624 billy1624 commented Nov 15, 2022

PR Info

Bug Fixes

  • The information_schema.columns.generaion_expression column is being added on MySQL 5.7, we'd skip select this column if the target database is MySQL 5.6 or before.

@billy1624 billy1624 self-assigned this Nov 15, 2022
@billy1624 billy1624 changed the title Run CI on MySQL 5.6 Backward compatible schema discovery for MySQL 5.6 Nov 15, 2022
@billy1624 billy1624 force-pushed the mysql-generation-expr branch from f62908f to 57d1674 Compare November 15, 2022 07:39
@billy1624 billy1624 force-pushed the mysql-generation-expr branch from 7009d82 to 91a802f Compare November 15, 2022 08:18
@tyt2y3
Copy link
Member

tyt2y3 commented Nov 15, 2022

As much as I wanted to, I think we can't put 5.6 on our CI going forward.
But nice to have a fix for now.

@tyt2y3
Copy link
Member

tyt2y3 commented Nov 15, 2022

So we can probably give up on fixing the CI, just make sure it still works on 5.latest and test it offline with 5.6
I guess there is a docker image of 5.6 available somewhere?

@billy1624
Copy link
Member Author

Yeah, I will revert the CI related changes now

@billy1624 billy1624 marked this pull request as ready for review November 15, 2022 08:45
@billy1624 billy1624 requested a review from tyt2y3 November 15, 2022 08:45
@billy1624
Copy link
Member Author

Ready

@tyt2y3
Copy link
Member

tyt2y3 commented Nov 16, 2022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants