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
schemadiff
INSTANT DDL: impossible changes on tables with FULLTEXT
index
#15725
schemadiff
INSTANT DDL: impossible changes on tables with FULLTEXT
index
#15725
Conversation
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
schemadiff
INSTANT DDL: impossible on tables with FULLTEXT
indexschemadiff
INSTANT DDL: impossible changes on tables with FULLTEXT
index
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15725 +/- ##
=======================================
Coverage 68.42% 68.42%
=======================================
Files 1559 1559
Lines 196482 196496 +14
=======================================
+ Hits 134438 134461 +23
+ Misses 62044 62035 -9 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
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.
LGTM! I can see that these limitations still exist in 8.4: https://dev.mysql.com/doc/refman/8.4/en/innodb-online-ddl-operations.html
Columns cannot be added to tables that use ROW_FORMAT=COMPRESSED, tables with a FULLTEXT index, tables that reside in the data dictionary tablespace, or temporary tables. Temporary tables only support ALGORITHM=COPY.
IMO it's worth noting that this PR is about tables with a FULLTEXT index OR that are using table compression.
True, I realized there was a missing check for compressed tables and added it while here. |
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Description
INSTANT
DDL analysis: one cannot runADD COLUMN
norDROP COLUMN
withINSTANT
algorithm on a table that has aFULLTEXT
index.Related Issue(s)
INSTANT
-capable changes #14877flavor
code. #14883Checklist
Deployment Notes