-
Notifications
You must be signed in to change notification settings - Fork 694
Docs: describe show create #19170
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
Docs: describe show create #19170
Conversation
⚪ Test history | Ya make output | Test bloat
🟢 |
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
❌ Documentation buildRevision build failed Build logsErrors (2)❌ ENOENT: no such file or directory, open '/en/yql/reference/syntax/show_create.md' ❌ ENOENT: no such file or directory, open '/ru/yql/reference/syntax/show_create.md' |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
❌ Documentation buildRevision build failed Build logsErrors (2)❌ ENOENT: no such file or directory, open '/en/yql/reference/syntax/show_create.md' ❌ ENOENT: no such file or directory, open '/ru/yql/reference/syntax/show_create.md' |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (3) |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
ADD CHANGEFEED `feed_3` WITH (MODE = 'KEYS_ONLY', FORMAT = 'JSON', RETENTION_PERIOD = INTERVAL('PT30M'), TOPIC_MIN_ACTIVE_PARTITIONS = 3) | ||
; | ||
|
||
ALTER SEQUENCE `/Root/my_table/_serial_column_Key3` START WITH 101 INCREMENT BY 404 RESTART; |
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.
Why is this the only place that uses an absolute path?
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.
Для alter sequence сейчас поддерживается только абсолютный путь
- [ALTER TABLE ... ALTER INDEX](alter_table/secondary_index#alter-index) — for index partitioning settings. | ||
- [ALTER TABLE ... ADD CHANGEFEED](alter_table/changefeed.md) — for adding a changefeed. | ||
- `ALTER SEQUENCE` — for restoring a `Sequence` state for `Serial` columns. | ||
- For views: the definition via [CREATE VIEW](create-view.md), and, if necessary, [PRAGMA TablePathPrefix](pragma#table-path-prefix). |
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.
The TablePathPrefix thing needs more clarification: when is it added, and why is it necessary?
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.
Unfortunately, the capturing of pragmas by views is not yet described in the documentation. Here is the issue:
I propose to not block the current PR and merge it with something minimal, like:
For views: the definition via CREATE VIEW and the statements the view has [captured](here would be the link to the context capturing by views) from the creation context, for example, PRAGMA TablePathPrefix.
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.
Добавил описание
Co-authored-by: Ivan Blinkov <ivan@blinkov.ru>
Co-authored-by: Ivan Blinkov <ivan@blinkov.ru>
🔄 New commits pushed — @anton-bobkov please take a look. |
Co-authored-by: Ivan Blinkov <ivan@blinkov.ru>
🔄 New commits pushed — @anton-bobkov please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (1)
|
🔄 New commits pushed — @anton-bobkov please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully |
🔄 New commits pushed — @anton-bobkov please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully |
Hey @anton-bobkov, it has been 29 business-hours since the author's last update, could you please review? |
Hey @anton-bobkov, it has been 59 business-hours since the author's last update, could you please review? |
Hey @anton-bobkov, it has been 89 business-hours since the author's last update, could you please review? |
Hey @anton-bobkov, it has been 119 business-hours since the author's last update, could you please review? |
Changelog entry
Changelog category
Description for reviewers
Docs for:
Show create table (Row-oriented tables) #14789
Fix show create table with darwin #15617
Update system view show_create #15825
Show create table (Column-oriented tables) #16509
Support CHANGEFEEDs in SHOW CREATE TABLE #17342
Support sequences in SHOW CREATE TABLE #17488
Support partition policy for Index Table in SHOW CREATE TABLE #17759
Support ALTER OBJECT for SHOW CREATE TABLE #18493
Fix ALTER OBJECT for SHOW CREATE TABLE #18531
fail gracefully on SHOW CREATE TABLE targeting a view #16423
refactoring before SHOW CREATE VIEW #16629
support SHOW CREATE VIEW #16668
refactoring in unit tests of SHOW CREATE TABLE #16929
SHOW CREATE VIEW: use YQL parser to split view query to context recreation and select parts #16931
Issues:
Support CHANGEFEEDs in SHOW CREATE TABLE #17074
Support sequences in SHOW CREATE TABLE #17350
Support partition policy for Index Table in SHOW CREATE TABLE #17680
view: show create #16283
Support SHOW CREATE TABLE returning SQL query text to recreate a table #33
show create view follow up: split view query using parser #16778
view: show create #16283
show create table: when targeting a view that reads from a table t, produces results of a 'select * from t' #16401