Skip to content

Enable the analyzer for table DDL#88087

Merged
alexey-milovidov merged 6 commits intoClickHouse:masterfrom
maxjustus:enable-analyzer-for-ddl
Oct 16, 2025
Merged

Enable the analyzer for table DDL#88087
alexey-milovidov merged 6 commits intoClickHouse:masterfrom
maxjustus:enable-analyzer-for-ddl

Conversation

@maxjustus
Copy link
Copy Markdown
Contributor

@maxjustus maxjustus commented Oct 3, 2025

Enable the analyzer for CREATE/ALTER TABLE column default expression validation

Changelog category (leave one):

  • Improvement

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

Enables the analyzer for CREATE/ALTER TABLE column default expression validation

@maxjustus maxjustus force-pushed the enable-analyzer-for-ddl branch from c3a68aa to 09a2d0b Compare October 3, 2025 22:00
@alexey-milovidov alexey-milovidov changed the title Enable new analyzer for table DDL Enable the analyzer for table DDL Oct 4, 2025
@alexey-milovidov alexey-milovidov added the can be tested Allows running workflows for external contributors label Oct 4, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Oct 4, 2025

Workflow [PR], commit [592092f]

Summary:

job_name test_name status info comment
Performance Comparison (amd_release, master_head, 2/3) failure
Start failure

@clickhouse-gh clickhouse-gh bot added the pr-improvement Pull request with some product improvements label Oct 4, 2025
@maxjustus maxjustus force-pushed the enable-analyzer-for-ddl branch 4 times, most recently from c123bbc to 3abd43b Compare October 8, 2025 21:47
@maxjustus
Copy link
Copy Markdown
Contributor Author

Ok this is green aside from the ytsaurus failure which looks like an unrelated issue that has an open pr to fix

@maxjustus maxjustus force-pushed the enable-analyzer-for-ddl branch from 3abd43b to 592092f Compare October 15, 2025 18:35
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov 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!

@alexey-milovidov alexey-milovidov self-assigned this Oct 16, 2025
@alexey-milovidov alexey-milovidov merged commit d686eaa into ClickHouse:master Oct 16, 2025
119 of 123 checks passed
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 16, 2025
@maxjustus maxjustus deleted the enable-analyzer-for-ddl branch October 16, 2025 22:27
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-improvement Pull request with some product improvements 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.

3 participants