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

Support table comment and column comment for view #1799

Merged
merged 8 commits into from Sep 18, 2019

Conversation

morningman
Copy link
Contributor

  1. Support table comment when creating table or view
  2. Support column comment when creating view.

See help create table and help create view for help.
Sett help show full columns and help show table status to view comment.

@kangkaisen
Copy link
Contributor

+1. LGTM. Please add a UT for this PR.

@morningman
Copy link
Contributor Author

+1. LGTM. Please add a UT for this PR.

OK

[BROKER PROPERTIES ("key"="value", ...)];
[PROPERTIES ("key"="value", ...)]
[BROKER PROPERTIES ("key"="value", ...)]
[COMMENT "table comment"];
Copy link
Contributor

Choose a reason for hiding this comment

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

To comply with mysql, comment should locate before partition define. Both Comment and Engine are table options. Here is the reference.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed

Copy link
Contributor

@imay imay left a comment

Choose a reason for hiding this comment

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

LGTM

@imay imay merged commit 714dca8 into apache:master Sep 18, 2019
@imay imay mentioned this pull request Sep 26, 2019
SWJTU-ZhangLei pushed a commit to SWJTU-ZhangLei/incubator-doris that referenced this pull request Jul 25, 2023
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.

None yet

3 participants