-
Notifications
You must be signed in to change notification settings - Fork 27
PS-10085 MyRocks limitations section still showing the Online DDL limitation part (8.0) #557
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
base: 8.0
Are you sure you want to change the base?
Conversation
d25fc52
to
09077ea
Compare
09077ea
to
357adc8
Compare
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.
Great.
357adc8
to
58be605
Compare
…itation part modified: docs/limitations.md
58be605
to
005192b
Compare
ea313d9
to
31e5846
Compare
31e5846
to
c8dac0f
Compare
| [`rocksdb_enable_instant_ddl_for_drop_index_changes=ON`](variables.md#rocksdb_enable_instant_ddl_for_drop_index_changes) | `ALTER TABLE ... DROP INDEX` | `INSTANT` | | ||
| [`rocksdb_enable_instant_ddl_for_table_comment_changes=ON`](variables.md#rocksdb_enable_instant_ddl_for_table_comment_changes) | `ALTER TABLE ... COMMENT` | `INSTANT` | | ||
|
||
**Note:** MyRocks does **not support `ALGORITHM=INSTANT`** in SQL syntax. These operations behave like Instant DDL **internally**, but only if the respective configuration variables are enabled. |
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.
This is not true anymore. ALGORITHM=INSTANT
is required for to use INSTANT DDLs e.g.:
1. rocksdb_enable_instant_ddl_for_table_comment_changes=ON
2. ALTER TABLE … COMMENT …, ALGORITHM=INSTANT;
Without ALGORITHM=INSTANT
, ALTER TABLE
will use default ALGORITHM
even if rocksdb_enable_instant_ddl_for_table_comment_changes = ON
.
With rocksdb_enable_instant_ddl_for_table_comment_changes=OFF
, using ALGORITHM=INSTANT
will cause error.
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.
modified: docs/limitations.md