Skip to content

Conversation

@GrigoryPervakov
Copy link
Member

Query failed after an attempt to create ColumnsDescription from the block with a duplicated column.

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@clickhouse-gh
Copy link

clickhouse-gh bot commented Sep 24, 2025

Workflow [PR], commit [4945f2e]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 24, 2025
@GrigoryPervakov GrigoryPervakov force-pushed the pervakovg/default_analyzer branch from 6d964ce to 0a6445a Compare September 24, 2025 17:49
@novikd novikd self-assigned this Sep 26, 2025
@GrigoryPervakov GrigoryPervakov force-pushed the pervakovg/default_analyzer branch from 0a6445a to 4945f2e Compare October 1, 2025 13:59
@GrigoryPervakov GrigoryPervakov marked this pull request as ready for review October 2, 2025 16:14
@novikd
Copy link
Member

novikd commented Oct 3, 2025

What has changed since the previous PR?

@GrigoryPervakov
Copy link
Member Author

GrigoryPervakov commented Oct 4, 2025

What has changed since the previous PR?

  • Fixed column duplication in the prewhere
  • Allow to disable analyzer in merge tree reader default expressions
  • Handle duplicated columns

Original commit is reverted untouched, all this changes are made as separate commits, so you can review it separately

@alexey-milovidov
Copy link
Member

Love it!

@alexey-milovidov alexey-milovidov merged commit c1eede5 into master Oct 16, 2025
122 of 123 checks passed
@alexey-milovidov alexey-milovidov deleted the pervakovg/default_analyzer branch October 16, 2025 17:43
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 16, 2025
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 pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants