Skip to content

Docs for ALTER ORDER BY#3755

Merged
alexey-milovidov merged 2 commits intomasterfrom
alter-order-by-docs
Dec 4, 2018
Merged

Docs for ALTER ORDER BY#3755
alexey-milovidov merged 2 commits intomasterfrom
alter-order-by-docs

Conversation

@ztlpn
Copy link
Copy Markdown
Contributor

@ztlpn ztlpn commented Dec 4, 2018

No description provided.

@alexey-milovidov alexey-milovidov merged commit d195902 into master Dec 4, 2018
Comment thread docs/en/operations/table_engines/mergetree.md
- Provide additional logic when data parts merging in the [CollapsingMergeTree](collapsingmergetree.md#table_engine-collapsingmergetree) and [SummingMergeTree](summingmergetree.md#table_engine-summingmergetree) engines.

You may need many fields in the primary key even if they are not necessary for the previous steps.
If you need this, it makes sense to specify the *sorting key* that is distinct from the primary key.
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.

If you need this -> In some cases ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This comment applies specifically to the case when the user needs additional merge logic (specified in the bullet point above).


<a name="table_engines-mergetree-sorting_key"></a>

### Choosing the Sorting Key that is distinct from the Primary Key
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.

"distinct" might get confused with keyword, I'd just say "...Sorting Key that Differs from The Primary Key"

Comment thread docs/en/query_language/alter.md
Comment thread docs/en/query_language/alter.md
@ztlpn
Copy link
Copy Markdown
Contributor Author

ztlpn commented Dec 5, 2018

@blinkov Thanks for the comments. 64bdb2a

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.

3 participants