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

Analyzer support recursive CTEs #62074

Merged

Conversation

kitaisreal
Copy link
Collaborator

Changelog category (leave one):

  • New Feature

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

Analyzer support recursive CTEs.

@kitaisreal kitaisreal added the can be tested Allows running workflows for external contributors label Mar 28, 2024
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-feature Pull request with new product feature label Mar 28, 2024
@robot-ch-test-poll3
Copy link
Contributor

robot-ch-test-poll3 commented Mar 28, 2024

This is an automated comment for commit c570316 with description of existing statuses. It's updated for the latest CI running

⏳ Click here to open a full report in a separate page

Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR⏳ pending
Successful checks
Check nameDescriptionStatus
A SyncThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS✅ success
Docs checkBuilds and tests the documentation✅ success
Fast testNormally this is the first check that is ran for a PR. It builds ClickHouse and runs most of stateless functional tests, omitting some. If it fails, further checks are not started until it is fixed. Look at the report to see which tests fail, then reproduce the failure locally as described here✅ success
Mergeable CheckChecks if all other necessary checks are successful✅ success
PR CheckThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS✅ success
Style checkRuns a set of checks to keep the code style clean. If some of tests failed, see the related log from the report✅ success

@novikd novikd added the analyzer Issues and pull-requests related to new analyzer label Apr 2, 2024
@novikd novikd self-assigned this Apr 2, 2024
@kitaisreal kitaisreal force-pushed the analyzer-support-recursive-cte branch from 6afea0f to 1593978 Compare April 6, 2024 18:02
@kitaisreal kitaisreal requested a review from novikd April 6, 2024 18:03
@kitaisreal kitaisreal force-pushed the analyzer-support-recursive-cte branch 6 times, most recently from 778672e to 5306f42 Compare April 10, 2024 12:53
@kitaisreal kitaisreal force-pushed the analyzer-support-recursive-cte branch 7 times, most recently from 3a0dea7 to 853a1f7 Compare April 27, 2024 15:52
@kitaisreal kitaisreal force-pushed the analyzer-support-recursive-cte branch from 853a1f7 to 56037ff Compare April 28, 2024 09:52
@alexey-milovidov alexey-milovidov added this pull request to the merge queue Apr 28, 2024
@alexey-milovidov alexey-milovidov removed this pull request from the merge queue due to a manual request Apr 28, 2024
@alexey-milovidov alexey-milovidov added this pull request to the merge queue Apr 28, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Apr 28, 2024
@alexey-milovidov alexey-milovidov added this pull request to the merge queue Apr 29, 2024
Merged via the queue into ClickHouse:master with commit dffcc51 Apr 29, 2024
11 of 19 checks passed
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Apr 29, 2024
@Algunenano
Copy link
Member

TSAN is not happy: https://s3.amazonaws.com/clickhouse-test-reports/0/dffcc51b8389734c7476b6baafa15c3535ada8d7/ast_fuzzer__tsan_/stderr.log

Let's fix it or revert before making a new release

@kitaisreal
Copy link
Collaborator Author

TSAN is not happy: https://s3.amazonaws.com/clickhouse-test-reports/0/dffcc51b8389734c7476b6baafa15c3535ada8d7/ast_fuzzer__tsan_/stderr.log

Let's fix it or revert before making a new release

I will try to make fix today.

@kitaisreal
Copy link
Collaborator Author

TSAN is not happy: https://s3.amazonaws.com/clickhouse-test-reports/0/dffcc51b8389734c7476b6baafa15c3535ada8d7/ast_fuzzer__tsan_/stderr.log

Let's fix it or revert before making a new release

Here is fix #63125.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer Issues and pull-requests related to new analyzer can be tested Allows running workflows for external contributors pr-feature Pull request with new product feature 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.

None yet

6 participants