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

Change column type based on lates Percona Toolkit documentation #201

Closed
wants to merge 1 commit into from

Conversation

ashatrov
Copy link

@ashatrov ashatrov commented Mar 12, 2019

@boxcla
Copy link

boxcla commented Mar 12, 2019

Hi @ashatrov, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla

Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

@ashatrov ashatrov changed the title Update install.sql Change column type based on lates Percona Toolkit documentation Mar 12, 2019
@koleo
Copy link

koleo commented Mar 14, 2019

@ashatrov you should first fix your PR: char(32) unsigned is not a valid datatype. You must remove the "unsigned" part.

However, it seems to me that this data type change requires some other fix in the PHP code...

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.

3 participants