Skip to content

support create table like statement#4111

Merged
sougou merged 9 commits intovitessio:masterfrom
klkyy2020:create-like
Jul 31, 2018
Merged

support create table like statement#4111
sougou merged 9 commits intovitessio:masterfrom
klkyy2020:create-like

Conversation

@klkyy2020
Copy link
Copy Markdown
Contributor

a small mr for supporting "create table xxx like xxx"
Signed-off-by: longfei klkyy2018@gmail.com

Signed-off-by: longfei <klkyy2018@gmail.com>
Signed-off-by: longfei <klkyy2018@gmail.com>
Copy link
Copy Markdown
Contributor

@sougou sougou left a comment

Choose a reason for hiding this comment

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

Could you also add a test in parse_test.go?
@demmer: do you think we should support this in vtexplain?

vindexParam VindexParam
vindexParams []VindexParam
showFilter *ShowFilter
OptLike *OptLike
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For uniformity, let's use optLike *OptLike. I know a few people have digressed, but we can fix those later.

Signed-off-by: longfei <klkyy2018@gmail.com>
Signed-off-by: longfei <klkyy2018@gmail.com>
@klkyy2020
Copy link
Copy Markdown
Contributor Author

updated

Signed-off-by: longfei <klkyy2018@gmail.com>
Signed-off-by: longfei <klkyy2018@gmail.com>
@sougou
Copy link
Copy Markdown
Contributor

sougou commented Jul 30, 2018

Looks good. I can merge once you resolve the merge conflicts.

klkyy2020 and others added 3 commits July 31, 2018 02:53
Signed-off-by: longfei <klkyy2018@gmail.com>
Signed-off-by: longfei <klkyy2018@gmail.com>
@klkyy2020
Copy link
Copy Markdown
Contributor Author

fixed :)

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.

2 participants