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.

Sign up for GitHub

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

Jump to bottom

Set index fix #4386

Merged
merged 9 commits into from
Feb 14, 2019
Merged

Set index fix #4386

merged 9 commits into from
Feb 14, 2019

Conversation

nikvas0
Copy link
Contributor

@nikvas0 nikvas0 commented Feb 13, 2019 •
edited
Loading

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

  1. Now set index requires exactly one argument (max stored rows number).
  2. Fixed bug in the set index. (dropping a granule if it contains more than max_rows rows)

@nikvas0 nikvas0 changed the title Set index args Set index fix Feb 14, 2019
@alexey-milovidov alexey-milovidov merged commit d5aba07 into ClickHouse:master Feb 14, 2019
@filimonov filimonov added the comp-skipidx Data skipping indices label May 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
comp-skipidx Data skipping indices
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nikvas0 @filimonov @alexey-milovidov