Skip to content

Planner: trivial count optimization#46923

Merged
kitaisreal merged 13 commits intomasterfrom
analyzer_trivial_count_optimization
Mar 7, 2023
Merged

Planner: trivial count optimization#46923
kitaisreal merged 13 commits intomasterfrom
analyzer_trivial_count_optimization

Conversation

@devcrafter
Copy link
Copy Markdown
Member

@devcrafter devcrafter commented Feb 26, 2023

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Related to #45461

@devcrafter devcrafter marked this pull request as draft February 26, 2023 22:53
@robot-ch-test-poll robot-ch-test-poll added the pr-not-for-changelog This PR should not be mentioned in the changelog label Feb 26, 2023
@devcrafter devcrafter marked this pull request as ready for review February 28, 2023 11:39
@devcrafter
Copy link
Copy Markdown
Member Author

devcrafter commented Feb 28, 2023

Make sense to review. Trivial count optimization with partition predicate can be implemented separately

Copy link
Copy Markdown
Contributor

@kitaisreal kitaisreal left a comment

Choose a reason for hiding this comment

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

Looks good to me. Need to fix a couple of comments and merge.

@devcrafter devcrafter requested a review from kitaisreal March 6, 2023 11:05
@kitaisreal kitaisreal merged commit 6cd4507 into master Mar 7, 2023
@kitaisreal kitaisreal deleted the analyzer_trivial_count_optimization branch March 7, 2023 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants