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

A bug fix for Planner #45822

Merged
merged 1 commit into from
Feb 1, 2023
Merged

A bug fix for Planner #45822

merged 1 commit into from
Feb 1, 2023

Conversation

taofengliu
Copy link
Contributor

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

When use experimental analyzer, queries like SELECT number FROM numbers(100) LIMIT 10 OFFSET 10; get wrong results (empty result for this sql).
That is caused by an unnecessary offset step added by planner.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-bugfix Pull request with bugfix, not backported by default label Jan 31, 2023
@taofengliu taofengliu mentioned this pull request Jan 31, 2023
1 task
@ucasfl ucasfl added the can be tested Allows running workflows for external contributors label Jan 31, 2023
@kitaisreal kitaisreal self-assigned this Jan 31, 2023
@devcrafter devcrafter merged commit 04c9fcd into ClickHouse:master Feb 1, 2023
@taofengliu taofengliu deleted the planner_bug branch February 1, 2023 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can be tested Allows running workflows for external contributors pr-bugfix Pull request with bugfix, not backported by default
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants