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

Removing online ddl query hint from ALTER TABLE #7069

Merged
merged 11 commits into from
Nov 25, 2020

Conversation

shlomi-noach
Copy link
Contributor

Context: #6782
Followup to #7045

This PR completely removed the WITH... syntax from ALTER TABLE and CREATE INDEX statements. Online DDL hints will now only be supported via:

  • @@ddl_strategy session variable on vtgate, or
  • -ddl_strategy command line flag on vtctl.

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach
Copy link
Contributor Author

Documentation PR to follow.

@shlomi-noach shlomi-noach marked this pull request as draft November 23, 2020 16:41
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
…-query-hint

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach
Copy link
Contributor Author

Ready for review. Documentation: vitessio/website#598

@shlomi-noach shlomi-noach marked this pull request as ready for review November 24, 2020 13:11
Copy link
Member

@deepthi deepthi left a comment

Choose a reason for hiding this comment

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

+1. This seems like a much cleaner approach than all the other options you explored.

@shlomi-noach shlomi-noach merged commit 3b94d7a into vitessio:master Nov 25, 2020
@shlomi-noach shlomi-noach deleted the remove-online-ddl-query-hint branch November 25, 2020 07:34
@askdba askdba added this to the v9.0 milestone Nov 27, 2020
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.

4 participants