Skip to content

Transform row-oriented table to columnar-oriented table - #311

Merged
imay merged 2 commits into
apache:masterfrom
chaoyli:master
Nov 16, 2018
Merged

Transform row-oriented table to columnar-oriented table#311
imay merged 2 commits into
apache:masterfrom
chaoyli:master

Conversation

@chaoyli

@chaoylichaoyli commented Nov 14, 2018

Copy link
Copy Markdown
Contributor

Apache Doris has not supported row-oriented table any more.
In future release, row-oriented table should be transformed to columnar-oriented table.
Now we provide a schema change operator to accomplish this transformation.
Also we provide a script to generate all command about transformation, it should be performed before upgrade.
Script is stored in docs/row_to_column/ directory.

@doris-cidoris-ci reopened this Nov 15, 2018
imay
imay requested changes Nov 15, 2018
View reviewed changes

@imayimay left a comment •
edited by chaoyli
Loading

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.

I have remove binary file from code.

Comment threadtools/row_to_column/get_ddl_stmt.py
Comment threadtools/row_to_column/pkg_resources.py Outdated
@chaoylichaoyli changed the title transform row-oriented table to columnar-oriented tableTransform row-oriented table to columnar-oriented tableNov 15, 2018
@doris-cidoris-ci reopened this Nov 15, 2018
@doris-ci

doris-ci commented Nov 15, 2018

Copy link
Copy Markdown

checks通过了

imay
imay requested changes Nov 16, 2018
View reviewed changes
Comment threadbe/src/olap/olap_engine.cpp Outdated
Comment threadbe/src/olap/olap_header_manager.cpp
Comment threadbe/src/olap/store.cpp Outdated
imay
imay approved these changes Nov 16, 2018
View reviewed changes

@imayimay left a comment

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.

LGTM

@imay
imay merged commit 1ba8a4e into apache:masterNov 16, 2018
@lide-reedlide-reed mentioned this pull request Feb 18, 2019
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@imayimayimay approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chaoyli@doris-ci@imay