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

Fix query_id in part_log with async flush queries #53103

Merged
merged 3 commits into from Aug 10, 2023

Conversation

Algunenano
Copy link
Member

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

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

  • Fix query_id in part_log with async flush queries

Documentation entry for user-facing changes

Ref #51160 (comment)

@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-bugfix Pull request with bugfix, not backported by default label Aug 7, 2023
@robot-ch-test-poll4
Copy link
Contributor

robot-ch-test-poll4 commented Aug 7, 2023

This is an automated comment for commit 232b2c0 with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟡 pending

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
Mergeable CheckChecks if all other necessary checks are successful🟢 success
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub🟢 success

@CurtizJ CurtizJ self-assigned this Aug 7, 2023
@CurtizJ
Copy link
Member

CurtizJ commented Aug 7, 2023

Test test_zookeeper_config is flaky

@CurtizJ
Copy link
Member

CurtizJ commented Aug 7, 2023

Test test_profile_max_sessions_for_user is more suspicious but also seems to be flaky

@CurtizJ CurtizJ merged commit c09870b into ClickHouse:master Aug 10, 2023
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-bugfix Pull request with bugfix, not backported by default
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants