-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Generated column as part of UNIQUE (or PRIMARY) KEY #919
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
Generated column as part of UNIQUE (or PRIMARY) KEY #919
Conversation
Actions/Workflow: upload artifact
Support a complete ALTER TABLE statement in --alter
handle driver timeout error
Updates from upstream 2020 10
Initial commit: towards setting up a test suite Signed-off-by: Shlomi Noach <[email protected]>
…original table, applying AUTO_INCREMENT value onto ghost table if applicable and user has not specified AUTO_INCREMENT in alter statement
Signed-off-by: Shlomi Noach <[email protected]>
Signed-off-by: Shlomi Noach <[email protected]>
Copying AUTO_INCREMENT value to ghost table
Signed-off-by: Shlomi Noach <[email protected]>
Signed-off-by: Shlomi Noach <[email protected]>
At this time this PR appears to contain many more commits than actually changed; this is because there's still other outstanding requests to merge upstream. For reference, the specific changes to this PR are https://github.com/openark/gh-ost/pull/13/files |
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.
Thanks @shlomi-noach 👍
@shlomi-noach 👋 this is being tested now! Could you re-base this branch so it's ready to merge after a few days of testing? 🙇 |
Rebased! |
Thanks @shlomi-noach this passed tests for over a week. Merging 👍 |
Resubmission of openark#13 from downstream.
Supporting the case where any unique key, and in particular the
PRIMARY KEY
, includes aGENERATED
column.A user with a large database deployment, who initially reported the issue, has confirmed the fix works as expected.
script/cibuild
returns with no formatting errors, build errors or unit test errors.