Skip to content

Analyzer planner fixes before enable by default#47383

Merged
kitaisreal merged 15 commits intoClickHouse:masterfrom
kitaisreal:analyzer-planner-fixes-before-enable-by-default
Mar 11, 2023
Merged

Analyzer planner fixes before enable by default#47383
kitaisreal merged 15 commits intoClickHouse:masterfrom
kitaisreal:analyzer-planner-fixes-before-enable-by-default

Conversation

@kitaisreal
Copy link
Copy Markdown
Contributor

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@kitaisreal kitaisreal requested a review from novikd March 8, 2023 16:25
@kitaisreal kitaisreal added the can be tested Allows running workflows for external contributors label Mar 8, 2023
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Mar 8, 2023
@novikd novikd self-assigned this Mar 8, 2023
@kitaisreal kitaisreal force-pushed the analyzer-planner-fixes-before-enable-by-default branch from 2d4aa23 to 4893488 Compare March 9, 2023 09:05
@den-crane
Copy link
Copy Markdown
Contributor

Sorry, not sure do you need it or not.
I found accidentally that a query does not work with the new analyzer
https://fiddle.clickhouse.com/fcc1dcb9-de2e-4233-ada3-b826a9ca364e

@kitaisreal kitaisreal force-pushed the analyzer-planner-fixes-before-enable-by-default branch from 9f4593b to e2a7cec Compare March 10, 2023 17:50
@novikd
Copy link
Copy Markdown
Member

novikd commented Mar 10, 2023

Sorry, not sure do you need it or not.
I found accidentally that a query does not work with the new analyzer
https://fiddle.clickhouse.com/fcc1dcb9-de2e-4233-ada3-b826a9ca364e

@den-crane works fine for me on master

@kitaisreal kitaisreal force-pushed the analyzer-planner-fixes-before-enable-by-default branch from e2a7cec to c835fa3 Compare March 11, 2023 10:52
@kitaisreal kitaisreal merged commit 114d30d into ClickHouse:master Mar 11, 2023
@kitaisreal kitaisreal deleted the analyzer-planner-fixes-before-enable-by-default branch March 15, 2023 16:20
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-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.

4 participants