Skip to content

ISSUES-4006 support first for ALTER ADD|MODIFY COLUMN#12073

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
zhang2014:feature/atfer_column_modify_column
Jul 3, 2020
Merged

ISSUES-4006 support first for ALTER ADD|MODIFY COLUMN#12073
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
zhang2014:feature/atfer_column_modify_column

Conversation

@zhang2014
Copy link
Copy Markdown
Contributor

@zhang2014 zhang2014 commented Jul 1, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • New Feature

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
#4006 Support ALTER TABLE ... [ADD|MODIFY] COLUMN ... FIRST

@blinkov blinkov added doc-alert pr-feature Pull request with new product feature labels Jul 1, 2020
@zhang2014
Copy link
Copy Markdown
Contributor Author

zhang2014 commented Jul 1, 2020

Maybe the execution of 'ALTER TABLE test_table MODIFY COLUMN nested_column.x FIRST' should be rejected ?

@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Jul 3, 2020

Maybe the execution of 'ALTER TABLE test_table MODIFY COLUMN nested_column.x FIRST' should be rejected ?

Yes, it's reasonable to forbid moving of nested columns.
BTW, do we support MODIFY COLUMN ... AFTER ...?
Upd. I see.

Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-feature Pull request with new product feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants